← Back to issue list

commands: cosmetic fixes to promote-bundle

View original Github issue

Metadata

Project
charmcraft
Number
#1094
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged cosmetic updates to the promote-bundle command. Channel parameters were converted to options, completion output was added, and an upload artifact was removed. Approved and merged despite an unrelated test failure tracked separately.

Suggested action:

No scores available.

Issue body

Make channel parameters options instead of positional arguments and display what was done at the end (which also gets rid of an upload artifact left over from the last progress message).

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged cosmetic updates to the promote-bundle command. Channel parameters were converted to options, completion output was added, and an upload artifact was removed. Approved and merged despite an unrelated test failure tracked separately.
qwen3.6-35b-a3b-mtp-q6 Merged cosmetic updates to promote-bundle that convert channel parameters to options, add completion output, and fix a leftover upload artifact. Merged despite an unrelated test failure, tracked in issue #1095.
qwen3.6-35b-a3b-mtp-q6 Merged cosmetic updates to promote-bundle. Channel parameters were converted to options, completion output was added, and a progress artifact was removed. Merged despite an unrelated test failure, tracked separately.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1089 store: fix issues with promote-bundle command charmcraft merged Merged a fix for the promote-bundle command that resolved test breakage. The update modified 5 files with 16 additions and 14 deletions. Approved by one reviewer, the changes were merged despite an unresolved comment flagging a potential lingering issue.
78%
#1044 command: add the initial promote-bundle command charmcraft merged Merged the initial promote-bundle command, adding 449 lines across 12 files to resolve CRAFT-1516. Approved by two reviewers and successfully integrated.
73%