config: support use actions in charmcraft.yaml
Metadata
Current evaluation
Merged update allowing actions in charmcraft.yaml instead of separate actions.yaml. Includes refactoring to fix circular imports. Approved by one reviewer across 10 files with 345 additions and 3 deletions.
Suggested action: —
No scores available.
Issue body
This allow we define actions in the same file rather than separated actions.yaml.
This also included some refactor to avoid circular imports.
CRAFT-1746
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged update allowing actions in charmcraft.yaml instead of separate actions.yaml. Includes refactoring to fix circular imports. Approved by one reviewer across 10 files with 345 additions and 3 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged, allowing actions to be defined directly in charmcraft.yaml instead of a separate actions.yaml file. Includes refactoring to eliminate circular imports. Resolves CRAFT-1746. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Enables action definitions directly in charmcraft.yaml, eliminating the need for a separate actions.yaml file. Includes refactoring to resolve circular imports. Addresses CRAFT-1746. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #36 'charmcraft build' should pull in actions.yaml and config.yaml | charmcraft | merged | 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. | |
| #31 feat: charmcraft actions | craft-actions | merged | Merged changes adding two GitHub Actions: charmcraft/setup installs Charmcraft and LXD, while charmcraft/pack builds charms. Approved by two reviewers with passing CI checks, the update adds 299 lines across six files to streamline workflows. | |
| #1126 Allow use actions/config/metadata in charmcraft.yaml | charmcraft | merged | Merged. Allows defining actions, config, and metadata directly in charmcraft.yaml when corresponding files are absent. Addresses CRAFT-1747, CRAFT-1788, and CRAFT-1789. Approved with passing CI checks. | |
| #142 fix the action generated by 'charmcraft init' | charmcraft | merged | Merged a fix for the action generated by charmcraft init, resolving issue #141. Approved by three reviewers and verified by the reporter. Corrects a generation bug in the init command. | |
| #49 ci: test the charmcraft actions in more places | craft-actions | merged | Merged after two approvals and passing all CI checks. The update expands the continuous integration pipeline to test charmcraft actions across additional build environments, modifying a single file with minor configuration changes. | |
| #1829 fix(reactive): don't overwrite actions.yaml for reactive charms | charmcraft | merged | Merged to fix issue #1816 by preventing the overwriting of actions.yaml for reactive charms. Approved by two reviewers, the change updates two files (+29/-5 lines) and resolves CRAFT-3247. | |
| #72 fix: use new workaround for using other actions | craft-actions | merged | Merged fix for #71 replacing a broken composite action workaround with a new method copying files to unique paths. Applied the same update to charmcraft actions per reviewer feedback. Approved and integrated. | |
| #1149 command: charmcraft init to use charmcraft.yaml only | charmcraft | merged | Merged after review. Updated charmcraft init to use a unified charmcraft.yaml by default, replacing separate metadata, actions, and config files. Added profile descriptions and updated URLs. Initially blocked but eventually merged. | |
| #1622 fix: reorganise charmcraft.yaml preprocessing | charmcraft | merged | Merged following two approvals and passing CI. Reorganized charmcraft.yaml preprocessing to apply extensions on raw data, assign default parts to charms with bases, and enforce dictionary type for parts. Modified 5 files (+27/-14 lines). | |
| #50 test: test the snapcraft actions | craft-actions | merged | Merged adding tests for snapcraft actions, mirroring charmcraft tests. Approved by two reviewers with all CI checks passing. Updates two files with 89 additions and 1 deletion. |