recording: record the original snapcraft.yaml
Metadata
Current evaluation
Merged to record the original snapcraft.yaml. Approved by one reviewer, modified 6 files with 91 additions and 31 deletions. Author noted the change requires PR #1406.
Suggested action: —
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to record the original snapcraft.yaml. Approved by one reviewer, modified 6 files with 91 additions and 31 deletions. Author noted the change requires PR #1406. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to record the original snapcraft.yaml during the recording process. The change was applied after satisfying the dependency on PR #1406. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to record the original snapcraft.yaml during execution. The change was resolved after being marked as dependent on PR #1406, successfully integrating the feature into the codebase. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1278 recording: save the snapcraft.yaml in the resulting snap | snapcraft | merged | Merged. Saves snapcraft.yaml to prime/snap in the resulting snap. Approved by one reviewer with passing CI and 100% diff coverage. Unresolved comments remain, but the implementation is accepted as a starting point. | |
| #58 changes to snapcraft.yaml, and version, for 0.2.0 | charmcraft | merged | Merged following single reviewer approval. Updated snapcraft.yaml and bumped project version to 0.2.0 across two files, adding 107 lines and removing 9. | |
| #204 Update snapcraft.yaml | charmcraft | merged | Merged updates to snapcraft.yaml by markshuttle. Applied text changes for snapcraft.io and snap users, approved by two reviewers with zero unresolved comments. Diff added 18 lines and removed 14 across one file. | |
| #2734 SnapcraftError refactoring | snapcraft | merged | Refactored SnapcraftError across four files, adding 279 lines and removing 26. Approved by one reviewer and merged into the codebase. | |
| #146 Syntax of snapcraft.yaml (1.x) | snapcraft | merged | Merged after rebasing to resolve diff conflicts and squashing commits. The pull request updates snapcraft.yaml (1.x) syntax and received final approval before integration. | |
| #1566 recording: record build-snaps installed during the pull | snapcraft | merged | Merged to record build-snaps installed during the pull step, addressing #1556. Approved by two reviewers with a +74/-8 line change across eight files. | |
| #2160 many: refactor snapcraft.yaml loading out of load_config | snapcraft | merged | Merged a refactor separating raw snapcraft.yaml loading from validated processing within load_config. Approved by reviewer, passed CI checks, and integrated into master with 95.54% diff coverage. | |
| #2238 reporting: record the released version for errors | snapcraft | merged | Merged changes to record the released snapcraft version in error reports. Approved by reviewer, passed CI checks, and integrated with a 13-line update. | |
| #1567 recording: record the snaps installed on the machine | snapcraft | merged | Merged after passing CI and receiving approval from two reviewers. The change records installed snaps on the machine and depends on PR #1566. Updates span 8 files with a net addition of 126 lines. | |
| #351 feat: add a default snapcraft.yaml | starbase | open | Adds a default snapcraft.yaml (+54 lines, 1 file) so users can build a snap via the craft-application flow. Approved by 1 reviewer but 8 review comments remain unresolved; last activity 462 days ago. |