recording: record build-snaps installed during the pull
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Updates: #1556
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes to record build-snaps installed during the pull step, resolving #1556. Updates align with snapd fake merge proposals in #1569. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updates #1556 to record build-snaps installed during the pull phase, ensuring build-time snap dependencies are tracked in the recording output. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1295 recording: record build packages installed in the snap | snapcraft | merged | Merged into master. Records build packages installed in the snap. Approved by two reviewers, changed 7 files (+146/-110 lines), and increased overall coverage by 0.04% with 100% diff coverage. | |
| #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. | |
| #1293 recording: record stage packages installed in the snap | snapcraft | merged | Merged into master. Records stage packages installed in the snap. Approved by one reviewer, modified 8 files, achieved 100% diff coverage, increased overall coverage by 0.04%, and depended on PR #1292. | |
| #1518 project: introduce build-snaps | snapcraft | merged | Merged after approval by two reviewers. Introduces build-snaps with advanced grammar support, modifying 24 files with +902/-10 lines. | |
| #1407 recording: record the original snapcraft.yaml | snapcraft | merged | 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. |