← 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 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.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Merged documentation updates to configure badges to only track the main branch, preventing in-progress PRs from showing false failures. The fix was applied and noted for upstreaming to starbase.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#193 docs: rework badges imagecraft merged Merged documentation badge updates. Removed broken badges and added new ones for codecov, ruff, and snap. Approved by three reviewers and passed all continuous integration checks.
79%
#555 docs(readme): add readthedocs build badge craft-parts merged Merged a maintainer-submitted update adding a ReadTheDocs build badge to the README. Approved by two reviewers, passed CI checks, and applied with minimal changes (+2/-0 lines).
74%
#4350 docs(readme): add readthedocs build badge snapcraft merged Merged adding a ReadTheDocs build status badge to the README. Approved by two reviewers, passed CI, modified one file (+4/-1), and left coverage unchanged.
74%
#103 docs: new link to docs craft-store merged Merged after approval. Updated README.md to add a documentation link and badge, resolving issue #101. The single-file change added three lines and removed one. The pull request passed review and CI checks without blocking the merge.
73%
#203 docs(readme): add badges starbase merged Merged. Added badges to the README. Initial sphinx-lint line-length failures for URLs were resolved after upstream starbase changes fixed the linting rules, allowing the documentation update to pass CI and merge.
72%
#31 docs: update main page with the standard template craft-archives merged Merged documentation update applying the standard template to the main page. Approved by one reviewer, passed CI checks, and added 12 lines to one file with no coverage impact.
71%
#4371 ci: fix weekly spread tests, add status badge snapcraft merged 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.
71%
#1104 docs: update links craft-parts merged Merged after two reviewers approved. Updated documentation to fix broken links. Integrated into the main branch despite some CI test failures.
70%