← Back to issue list

cli: Move to docopt, help cmd migrated.

View original Github issue

Metadata

Project
snapcraft
Number
#135
Type
pull request
State
closed
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

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.

Resolution: closed superseded

Reason: PR #135 was closed without merging after it was reworked; the identical 'cli: Move to docopt, help cmd migrated' change landed via merged PR #136 in commit 00642d304, which absorbed the review feedback. The maintainer comment disabling Travis for examples/integration tests applied to this same docopts transition series.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The help command has been migrated. Tests other than unit have been disabled until the transition is complete. Signed-off-by: Sergio Schvezov sergio.schvezov@canonical.com

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded 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.
stealth/space-bunny-alpha — closed superseded Closed without merge; the docopt help-command migration was completed by merged canonical/snapcraft#136.
qwen/qwen3.6-35b-a3b — — Closed without merge. The CLI help command migration to docopt was abandoned after the author disabled Travis due to test breakage. No review or CI checks were completed.
qwen3.6-35b-a3b-mtp-q6 — — Closed following the help command migration to docopt. Integration tests were temporarily disabled due to expected breakage during the transition.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/snapcraft#136 (confidence 97%)

    Merged PR #136, commit 00642d304, carries the same title/commit message and completes the docopt help-command migration that #135 proposed.

  • Related To: canonical/snapcraft#287 (confidence 45%)

    Follow-up merged PR #287 (commit 8eb915084) uses docopt for example/integration test arguments, finishing the test-side of the transition that #135 left disabled.

Related issues

Issue Project State Summary Similarity
#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.
87%
#142 cli: clean migrated to docopts snapcraft closed Closed unmerged and superseded by canonical/snapcraft#143, which landed the clean-command docopts migration in commit c947ec280.
80%
#144 cli: init cmd migrated to docopts. snapcraft merged PR #144 by sergiusens merged the `snapcraft init` CLI command onto the docopts-based command framework (commit 2ed68dde2, 4 files, +104/-63), part of the incremental docopts migration series.
75%
#1582506 --help needs some love snapcraft (launchpad) closed Resolved by snapcore/snapcraft#1307 (commit d34dc16af), replacing docopt with Click and adding clearer per-subcommand help and option summaries.
74%
#143 cli: clean cmd migrated to docopts. snapcraft merged Merged PR canonical/snapcraft#143 (commit c947ec280) migrated the clean CLI command to docopts and moved load_config into the yaml module, after positive review from kyrofa and come-maiz.
73%
#1533395 Use docopt to handle example tests arguments snapcraft (launchpad) closed Example tests were migrated to docopt by merged PR canonical/snapcraft#287 (commit 8eb915084; merge 9e783579b).
73%
#160 cli: snap command migrated to docopts snapcraft merged Merged the snap command's docopts migration in commit 3331c627c; directory-snap integration testing was explicitly deferred.
72%
#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.
71%
#145 cli: pull command migrated to docopts snapcraft merged Merged the pull CLI command's migration to docopts in commit 7f09faa72 after incorporating reviewer feedback.
71%