← Back to issue list

Include config.yaml in built charm

View original Github issue

Metadata

Project
charmcraft
Number
#31
Type
pull request
State
closed
Author
charness
Labels
Created
Updated
Closed

Current evaluation

The maintainer cherry-picked the relevant commit and merged it as a separate pull request (#60). After contributor agreement clearance, the change to include config.yaml in the built charm was successfully integrated.

Suggested action:

No scores available.

Issue body

Include config.yaml in the zipped charm if present in the source. Test both with and without a config.yaml in the simple-build case.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b The maintainer cherry-picked the relevant commit and merged it as a separate pull request (#60). After contributor agreement clearance, the change to include config.yaml in the built charm was successfully integrated.
qwen3.6-35b-a3b-mtp-q6 Cherry-picked after CLA clearance and closed. The config.yaml inclusion change was merged via PR #60, superseding this pull request.
qwen3.6-35b-a3b-mtp-q6 Cherry-picked and merged into pull request #60 following contributor agreement clearance. The initial commit adding config.yaml to the built charm was adopted, resolving the request while PR #36 addressed the core issue.

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.
74%