Provide the building charms functionality through the pack command.
Metadata
Current evaluation
Merged into main: charm building is now available through the pack command, in merge commit ebc0d75b.
Resolution: closed resolved
Reason: The approved implementation in PR #324 was merged into main as ebc0d75b, adding charm-building functionality to the pack command.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged into main: charm building is now available through the pack command, in merge commit ebc0d75b. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged into main: charm building was added to the pack command in canonical/charmcraft#324, through merge commit ebc0d75b. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Implements building charms functionality via the pack command. Approved by three reviewers, modifying four files (+219/-30 lines). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Building charms functionality is now exposed via the pack command. The pull request was merged into the codebase by maintainer facundobatista without additional comments. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #148 feat(charm): add a build planner for bases charms | craft-platforms | merged | Merged PR #148 added a Charm build planner for legacy charms using the bases key, addressing CHARMCRAFT-630 in commit 16ff218. | |
| #452 build: create charm builder tool (CRAFT-373) | charmcraft | closed | Superseded by merged charmcraft#453, which moved charm-building logic to a standalone CharmBuilder; the venv and pip optimizations were deferred. | |
| #1791 Charmcraft 3 including previously-built charms in new charm builds | charmcraft | closed | Fixed in PR #1793 (commit affc209b): subsequent charmcraft pack runs no longer include previously built .charm artifacts in the new charm. |