ci: fix weekly spread tests, add status badge
Metadata
Current evaluation
Merged changes fixing a typo in scheduled spread tests and adding a CI status badge. Approved by two reviewers, passed CI checks, and modified two files.
Suggested action: —
No scores available.
Issue body
- [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] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
The scheduled spread tests have never run successfully due to a minor typo.
The badge can be previewed here: https://github.com/snapcore/snapcraft/blob/weekly-spread/README.md
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes fixing a typo in scheduled spread tests and adding a CI status badge. Approved by two reviewers, passed CI checks, and modified two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for a typo that prevented weekly spread tests from running, and added a CI status badge to the README. Changes passed linting and unit tests before integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for a typo breaking weekly spread tests and added a CI status badge. The update resolved scheduled test failures and updated the repository README. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2032 ci: fixes to several weekly spread tests | charmcraft | merged | Merged CI fixes for several weekly spread tests. Approved by two reviewers and passed all checks with minimal changes across three files. Addresses initial test failures as a starting point for broader improvements. | |
| #2786 fix: update Weekly Spread badge to reference spread-manual.yaml | charmcraft | merged | Merged fix updating the README Weekly Spread badge and link from the removed spread-large.yaml to spread-manual.yaml. Approved by two reviewers, passed CI, and resolved issue #2755. | |
| #1600 test(spread): move flaky parallel-build test to weeklies | charmcraft | merged | Merged to address issue #1332 by relocating a flaky parallel-build test to the weekly suite. Approved by two reviewers, the change modifies two files (+19/-2) to stabilize CI runs. | |
| #127 Fix spread tests | imagecraft | merged | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). | |
| #1413 build: rename weekly spread test | charmcraft | merged | Merged after two reviewers approved. Renamed the weekly spread test in the build configuration, requiring #1410. Changes were minimal across two files. | |
| #3662 tests: update spread url | snapcraft | merged | Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage. | |
| #280 ci, spread: add "large spreads" suite | rockcraft | merged | Merged addition of a new large spreads test suite to CI. Configured to run weekly via cron or manually via label. Approved by two reviewers and successfully integrated. | |
| #4529 Spread update | snapcraft | merged | Merged after approval by two reviewers and passing CI checks. The update modifies spread configuration across three files, adding 36 and removing 40 lines. Code coverage tests passed. | |
| #4715 ci: adjust tags for spread runs | snapcraft | merged | Merged. Adjusted CI tags for spread runs to support Canonical's self-hosted runners. Approved by two reviewers, passed CI, and maintained coverage. Modified two files with six additions and six deletions. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged a workaround for a flaky spread test (issue #533). The change adjusts one file (+2/-1 lines) to skip a no-op name query check while retaining other validations. Approved by two reviewers and passed all CI checks. |