'charmcraft build' should pull in actions.yaml and config.yaml
Metadata
Current evaluation
Merged PR #36 as commit a8cdb744, making charmcraft build automatically include actions.yaml and config.yaml in charm builds.
Resolution: closed resolved
Reason: PR #36 was approved and merged, resolving the missing actions.yaml and config.yaml inclusion with commit a8cdb744.
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 PR #36 as commit a8cdb744, making charmcraft build automatically include actions.yaml and config.yaml in charm builds. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #36 via commit a8cdb744, adding automatic inclusion of actions.yaml and config.yaml to charmcraft builds. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after two reviewer approvals. The pull request implements automatic inclusion of actions.yaml and config.yaml during charmcraft builds, adding 80 lines across two files. Changes were successfully integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Updates charmcraft build to automatically package actions.yaml and config.yaml. The change was merged into the main branch by contributor chipaca without additional discussion. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #35 charmcraft doesn't support actions | charmcraft | closed | Reported that charmcraft builds omitted actions.yaml, so charms with actions failed on install with 'CharmEvents' has no attribute. Fixed by merged PR #36 (commit a8cdb744), which makes 'charmcraft build' copy actions.yaml and config.yaml into the charm. | |
| #1114 config: support use actions in charmcraft.yaml | charmcraft | merged | Merged #1114 as commit 201b40f6, enabling actions in charmcraft.yaml and refactoring configuration models to avoid circular imports. | |
| #31 feat: charmcraft actions | craft-actions | merged | Merged in commit 91513c1, adding charmcraft/setup to install Charmcraft and LXD and charmcraft/pack to run setup and package charms. | |
| #249 `charmcraft build` will let you build a charm that won't deploy | charmcraft | closed | Closed by facundobatista as a duplicate, tracking all build-time sanity checks (incl. config.yaml/actions.yaml validity) in the umbrella issue charmcraft#27; later implemented by merged PR #556 (a585aec6). | |
| #768 Include an 'actions' dir if present. | charmcraft | merged | Merged PR #768 (commit 5ada563c) to include an existing actions directory in packaged charms by default, fixing #764. | |
| #763 charmcraft V1.5.0 and V1.6.0 not bundling up bash actions | charmcraft | closed | Reported that charmcraft 1.5.0/1.6.0 omitted the charm's `actions` directory from built charms. Maintainer facundobatista closed it as a duplicate of #764, which was fixed by merged PR #768 (commit 5ada563c). | |
| #1126 Allow use actions/config/metadata in charmcraft.yaml | charmcraft | merged | Merged in canonical/charmcraft#1126 (044b94f8), adding inline actions, config, and ST087-compliant metadata support when standalone YAML files are absent. | |
| #31 Include config.yaml in built charm | charmcraft | closed | PR closed unmerged: #36 (commit a8cdb744) already added config.yaml to the built charm; maintainer cherry-picked this PR's separate test commit d6d5106 into #60 (e1944189, merged). | |
| #4 feat: add charmcraft build plans | craft-platforms | merged | Merged charmcraft build-plan support and supporting build-plan utilities in canonical/craft-platforms#4 via commit efa325b. | |
| #49 ci: test the charmcraft actions in more places | craft-actions | merged | Merged PR #49 in commit a8e5310, expanding Charmcraft action CI coverage across more Ubuntu architectures, channels, and pinned-revision runs. |