Skip to content

ci: remove dev branch jobs from daily workflow - #549

Merged
mafredri merged 2 commits into
masterfrom
mafredri/ci-maintenance
Oct 23, 2026
Merged

mafredri merged 2 commits into
masterfrom
mafredri/ci-maintenance

Conversation

@mafredri

@mafredri mafredri commented Dec 16, 2025 •

Copy link
Copy Markdown
Member

The daily workflow's bench-dev and test-dev jobs check out the dev branch, which no longer exists. They fail on every run, so the daily workflow reports failure even when the master jobs pass.

ci/static.sh is no longer part of this PR. Nothing on master calls it, and it reads ci/out/profile.txt, which only the ci/test.sh changes in #539 write, so it stays in #539.

Refs #539

🤖 This PR was created with the help of Coder Agents, and will be reviewed by a human. 🏂🏻

@mafredri
mafredri requested a review from mtojek December 16, 2025 18:59
@mafredri
mafredri enabled auto-merge (squash) December 16, 2025 18:59
@mafredri mafredri changed the title chore(ci): remove dev branch jobs and add static.sh ci: remove dev branch jobs and add static.sh Dec 16, 2025
@mafredri
mafredri disabled auto-merge December 16, 2025 19:13
@mafredri
mafredri enabled auto-merge (squash) December 16, 2025 19:14
Comment thread ci/static.sh Outdated
Comment thread ci/static.sh Outdated
Comment thread ci/static.sh Outdated
Remove bench-dev and test-dev jobs from daily workflow since the dev
branch no longer exists.

Add ci/static.sh script to generate coverage badge for GitHub Pages.

Refs #539
Nothing calls ci/static.sh, and it reads ci/out/profile.txt, which
ci/test.sh does not write (it writes ci/out/coverage.prof). The script
only works with the ci/test.sh changes in #539, so it moves back there
and this PR only drops the dev branch jobs.
@mafredri
mafredri force-pushed the mafredri/ci-maintenance branch from 5e27de9 to e7e5f43 Compare October 23, 2026 14:39
@mafredri mafredri changed the title ci: remove dev branch jobs and add static.sh ci: remove dev branch jobs from daily workflow Oct 23, 2026
@mafredri

Copy link
Copy Markdown
Member Author

@mtojek I removed ci/static.sh from this PR because it only works with the ci/test.sh changes in #539, where the same script still lives, so your notes apply there. I'm resolving the threads here since the file is gone.

🤖 This review was automatically generated with Coder Agents.

@mafredri
mafredri merged commit 325114f into master Oct 23, 2026
7 of 8 checks passed
@mafredri
mafredri deleted the mafredri/ci-maintenance branch October 23, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants