← Back to issue list

cli: stage command migrated to docopts

View original Github issue

Metadata

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

Current evaluation

Migrated the stage command to docopts. Resolved reviewer feedback regarding execution loop logic and test coverage. Approved by maintainers and merged.

Suggested action:

No scores available.

Issue body

What differs from the previous implementation is that the looping is done on the `steps` first and then on the `parts`, this makes the fact that `after` handling needs fixing right now instead of later as a _part_ depending on another needs to have its assets `staged`.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Migrated the stage command to docopts. Resolved reviewer feedback regarding execution loop logic and test coverage. Approved by maintainers and merged.
qwen3.6-35b-a3b-mtp-q6 Migrated the stage command to docopts and restructured loops to correctly stage dependent parts. Incorporated reviewer feedback to improve test coverage and logic. Approved and merged.
qwen3.6-35b-a3b-mtp-q6 Migrated the stage command to docopts, reordering the execution loop to handle part dependencies. Reviewers approved after feedback was addressed, with additional test coverage deferred to future branches. The changes were merged.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#149 cli: build command migrated to docopts snapcraft merged Migrated the CLI build command to docopts. The pull request was merged after reviewer approval, noting a minor copy-paste error. Changes spanned 3 files with 164 additions and 8 deletions.
72%
#144 cli: init cmd migrated to docopts. snapcraft merged Successfully merged the migration of the CLI init command to docopts. The change updated 4 files, passed CI checks, and received reviewer approval.
71%