chore: spread cleanup
Metadata
Current evaluation
Merged by maintainer sergiusens after three reviewers approved. The PR cleaned up spread configuration across 85 files, removing 358 lines and adding 115. All CI checks passed with 100% coverage on modified code.
Suggested action: —
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged by maintainer sergiusens after three reviewers approved. The PR cleaned up spread configuration across 85 files, removing 358 lines and adding 115. All CI checks passed with 100% coverage on modified code. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a maintainer-led chore PR for spread cleanup. Codecov confirmed all modified lines are covered with 89.16% project coverage. The changes were successfully integrated into the feature/craft-application branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer sergiusens. Spread configuration cleanup passed all automated checks, achieving 100% test coverage on modified lines. Changes were successfully integrated into the feature/craft-application branch. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1575 chore: make spread config a bit nicer | charmcraft | merged | Merged maintainer lengau's pull request to improve spread configuration formatting. The change added four lines across two files. Approved by two reviewers with zero unresolved comments and no CI checks required. | |
| #4674 chore(spread): miscellaneous spread fixes for core24 | snapcraft | merged | Merged following approval from two reviewers and successful code coverage checks. The change applies miscellaneous spread fixes for core24 across four files, resulting in a +9/-5 line diff. | |
| #4668 chore(spread): core22 provider setup cleanup | snapcraft | merged | Merged cleanup of the core22 provider setup in spread configuration. Fixes the cleanup process to target the correct provider. Approved by two reviewers, passed CI, and achieved full test coverage on modified lines across 12 files. | |
| #753 chore(services/testing): reduce spread verbosity level | craft-application | merged | Merged changes that disable verbose mode for the spread test command to reduce output noise. Approved by reviewers, passed all CI checks, and integrated into the codebase. | |
| #4689 chore(spread): remove remove hook test for core24 | snapcraft | merged | Merged after two reviewers approved and CI passed. The change removes a spread test for core24 base instance removal, as it was unrelated to core24 enablement. Two files were modified, reducing the codebase by 58 lines. | |
| #6010 chore!: remove remote-build service override | snapcraft | merged | Merged cleanup removing the remote-build service override and manual spread tests. This follow-up addresses overlooked work from PR #5994. Approved by two reviewers, modifying seven files. | |
| #251 chore(spread): add a multipass backend for spread | imagecraft | merged | Merged changes adding a multipass backend for spread, adapted from Charmcraft. The single-file update added 50 lines, received approval from two reviewers, and passed CI checks before integration. | |
| #5030 chore(merge): 8.4.0 into main | snapcraft | merged | Merged version 8.4.0 into main. Approved by two reviewers with passing CI checks. The update modified 33 files, adding 930 lines and removing 151. A maintainer clarified an unrelated launchpad 503 error triggered a core20 spread test failure. | |
| #603 chore: update spread test to use utility functions | fetch-service | merged | Merged maintainer PR replacing direct curl calls in spread tests with utility functions from spread/utils.sh and extending create_session to accept optional secret definitions. Approved by two reviewers with all CI checks passing. | |
| #907 chore(merge): 2.1.3 into main | craft-parts | merged | Merged version 2.1.3 into main following approval from two reviewers and passing CI checks. The change updated seven files, adding 36 lines and removing 10. |