feat(versions): include versions to CLI methods - #89
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
There was a problem hiding this comment.
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 versionsinto a command group withlistandshowsubcommands, including argument parsing/validation forversionNumber. - 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.
6f69335 to
66db731
Compare
No description provided.