copy: add slug_perm to output - #36
Conversation
|
Code Climate has analyzed commit bda37d7 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 0.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 1.2% (0.0% change). View more on Code Climate. |
|
Clones removed
==============
+ cloudsmith_cli/cli/commands/copy.py -1
See the complete overview on Codacy |
When using the copy command to copy packages between repositories, the unique ID (slug_perm) of the package wasn't being printed to stdout. As with other commands (such as 'push') which did, the output of this could then be parsed to further operate on the specific package. Therefore, make the output from 'copy' behave in the same way.
Co-authored-by: Cursor <cursoragent@cursor.com>
a8de853 to
c7ee1ac
Compare
There was a problem hiding this comment.
Pull request overview
Adds the package unique ID (slug_perm) to cloudsmith copy success output so downstream tooling can reliably identify the copied package, aligning behavior with push and other package-creation flows.
Changes:
- Capture and surface
slug_permreturned by the API when copying a package. - Print a human-readable
Copied: owner/repo/slug (slug_perm)success line (stderr in JSON modes, stdout otherwise). - Include
slug_permin-F json/-F pretty_jsonstatus output and document the change in the changelog.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| cloudsmith_cli/cli/commands/copy.py | Prints slug_perm after successful copy and includes it in JSON status output. |
| CHANGELOG.md | Documents the updated cloudsmith copy output behavior under Unreleased. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
When using the copy command to copy packages between repositories, the unique ID
(slug_perm) of the package wasn't being printed to stdout. As with other commands
(such as 'push') which did, the output of this could then be parsed to further operate
on the specific package.
Therefore, make the output from 'copy' behave in the same way.