Skip to content

feat(versions): include versions to CLI methods - #89

Merged
Ryank90 merged 1 commit into
feat/runserv-515from
feat/runserv-544
Aug 18, 2026
Merged

feat(versions): include versions to CLI methods#89
Ryank90 merged 1 commit into
feat/runserv-515from
feat/runserv-544

Conversation

@Ryank90

@Ryank90 Ryank90 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. To trigger a review, include coderabbit-review in the PR description. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5517fc55-af79-48ee-859e-ef0222398c0f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR expands the serverless “apps versions” surface area to support both listing versions and showing a single immutable version, aligning the CLI structure with the existing “builds” command pattern and adding the needed API client method.

Changes:

  • Refactors serverless apps versions into a command group with list and show subcommands, including argument parsing/validation for versionNumber.
  • Adds Client.GetVersion(...) to the serverless API client plus tests for success, not-found, and missing API key.
  • Extends tabular output support with a single-version display shape and updates generated docs accordingly.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
internal/cmd/serverless/display.go Adds versionResult for single-version table/json/yaml output.
internal/cmd/serverless/display_test.go Adds/adjusts tests for version display and versionNumber parsing.
internal/cmd/serverless/apps.go Removes the old leaf versions command implementation (now provided by new file).
internal/cmd/serverless/apps_versions.go Introduces versions group with list and show subcommands and parseVersionNumber.
internal/api/serverless/client.go Adds GetVersion API client method with status-code handling and debug logging.
internal/api/serverless/client_test.go Adds GetVersion test coverage (OK / 404 / no API key).
docs/runware_serverless_apps.md Updates the apps command list description for the versions command.
docs/runware_serverless_apps_versions.md Updates versions to a parent command and links to list/show.
docs/runware_serverless_apps_versions_show.md Adds docs page for versions show.
docs/runware_serverless_apps_versions_list.md Adds docs page for versions list.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Ryank90
Ryank90 marked this pull request as ready for review August 18, 2026 18:39
@Ryank90
Ryank90 merged commit 0dac279 into rc/serverless Aug 18, 2026
4 checks passed
@Ryank90
Ryank90 deleted the feat/runserv-544 branch August 18, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants