← Back to issue list

cli: pull command migrated to docopts

View original Github issue

Metadata

Project
snapcraft
Number
#145
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged the pull CLI command's migration to docopts in commit 7f09faa72 after incorporating reviewer feedback.

Resolution: closed resolved

Reason: PR #145 was merged as commit 7f09faa72. The author confirmed that all reviewer comments were addressed before approval.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Sergio Schvezov sergio.schvezov@canonical.com

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged the pull CLI command's migration to docopts in commit 7f09faa72 after incorporating reviewer feedback.
stealth/space-bunny-alpha — closed resolved Merged PR #145 migrated Snapcraft’s pull CLI command to docopts, incorporating reviewer feedback; landed as commit 7f09faa72.
stealth/space-bunny-alpha — closed resolved Merged canonical/snapcraft#145, migrating Snapcraft’s pull CLI command to docopts, as commit 7f09faa72 after incorporating reviewer feedback.
qwen/qwen3.6-35b-a3b — — Merged the CLI pull command migration to docopts. Author sergiusens incorporated reviewer feedback, resulting in a +238/-265 line change across 9 files. The changes were successfully integrated.
qwen3.6-35b-a3b-mtp-q6 — — Migrated the CLI pull command to docopts. Incorporated reviewer feedback and a minor typo fix. Successfully merged after maintainer approval.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#151 cli: strip command migrated to docopts snapcraft merged Merged PR #151, migrating Snapcraft's strip CLI command to docopts in commit 6251b86f3 across six files.
74%
#149 cli: build command migrated to docopts snapcraft merged Merged the build command's migration to docopts in commit 84aa15f34, changing 3 files (+164/-8); @come-maiz approved it with one copy-paste correction.
72%
#135 cli: Move to docopt, help cmd migrated. snapcraft closed Closed unmerged and superseded by canonical/snapcraft#136, which landed the same docopt help-command migration in commit 00642d304. Review feedback was addressed there; tests stayed disabled pending the wider CLI migration.
71%
#136 cli: Move to docopt, help cmd migrated. snapcraft merged Merged PR #136 (commit 00642d304), migrating the CLI to docopt and the help command to the new framework; non-unit tests were disabled pending the full transition.
70%