← Back to issue list

docs: make badges only use what's actually on main

View original Github issue

Metadata

Project
charmcraft
Number
#1505
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit 560436f0. Added ?event=push to the tests.yaml and spread.yaml badge URLs in README.md so badges reflect only main-branch runs, not in-flight PRs.

Resolution: closed resolved

Reason: PR #1505 was merged into main as commit 560436f0 ("docs: make badges only use what's actually on main"), a +2/-2 change to README.md. The ?event=push query param is still present on the tests and spread badge links at the current HEAD, confirming the change shipped.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Currently PRs to main are included in those badges, meaning that if tests fail on those PRs it shows failing.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit 560436f0. Added ?event=push to the tests.yaml and spread.yaml badge URLs in README.md so badges reflect only main-branch runs, not in-flight PRs.
stealth/space-bunny-alpha — closed resolved Merged PR #1505 in commit 560436f0, restricting README status badges to main so open PR test failures no longer appear as main failures.
qwen/qwen3.6-35b-a3b — — Merged a documentation update restricting status badges to the main branch only. This prevents open pull requests with failing tests from incorrectly displaying as failures. Approved by three reviewers, passed CI, and modified one file.
qwen3.6-35b-a3b-mtp-q6 — — Merged update configuring project badges to track only the main branch, preventing open PRs with failing tests from skewing status displays. Clarified branch filtering and noted upstreaming to starbase.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/starbase#180 (confidence 80%)

    mr-cal suggested in a comment on this PR that the badges should be upstreamed into starbase; that issue was later closed without an implementation.

Related issues

Issue Project State Summary Similarity
#193 docs: rework badges imagecraft merged PR #193 merged as commit dea806e ("docs: rework badges"): dropped broken README badges and added Codecov coverage, Ruff and Snap Store badges, +16/-8 lines in README.md.
72%