← Back to issue list

ci: fix weekly spread tests, add status badge

View original Github issue

Metadata

Project
snapcraft
Number
#4371
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged #4371 in commit 93867b692, fixing a typo that blocked scheduled spread tests and adding a workflow status badge.

Resolution: closed resolved

Reason: PR #4371 was merged as commit 93867b692, correcting the scheduled spread-test workflow and adding a status badge to document the fix.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged #4371 in commit 93867b692, fixing a typo that blocked scheduled spread tests and adding a workflow status badge.
stealth/space-bunny-alpha — closed resolved Merged PR #4371 in commit 93867b692 fixed a typo preventing scheduled weekly Spread tests from running and added a status badge.
stealth/space-bunny-alpha — closed resolved Merged PR #4371 in commit 93867b692, fixing a typo that blocked scheduled weekly spread tests and adding a status badge.
stealth/space-bunny-alpha — closed resolved Merged PR #4371 in commit 93867b692, fixing the scheduled spread-test typo and adding a CI status badge.
stealth/space-bunny-alpha — closed resolved PR #4371 merged in commit 93867b692, fixing the typo that prevented scheduled spread tests from running and adding a CI status badge.
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.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2786 fix: update Weekly Spread badge to reference spread-manual.yaml charmcraft merged Merged PR canonical/charmcraft#2786 (approved, 2 reviewers) fixed a stale README link: the Weekly Spread badge and results link were repointed from the deleted .github/workflows/spread-large.yaml to spread-manual.yaml, closing #2755.
78%
#2032 ci: fixes to several weekly spread tests charmcraft merged Merged PR canonical/charmcraft#2032 as commit 29cb07f3, applying a first pass of fixes to several weekly Spread tests.
74%
#1413 build: rename weekly spread test charmcraft merged Merged PR #1413 renamed the weekly Spread test and related references, landing as commit e1d73542 after prerequisite #1410.
74%
#469 docs(readme): add status badge for weekly tests craft-providers closed Closed unmerged as the README update was superseded by PR #470, which added weekly-test and coverage badges and merged as commit a5fc284.
74%
#1600 test(spread): move flaky parallel-build test to weeklies charmcraft merged Merged in commit a1d285c7, moving the flaky parallel-build Spread test from regular CI to weeklies as a workaround for #1332.
72%