Skip to content

fix: replace dead pages-build-deployment badge - #542

Merged
tschm merged 1 commit into
mainfrom
fix/readme-pages-badge
Aug 30, 2026
Merged

fix: replace dead pages-build-deployment badge#542
tschm merged 1 commit into
mainfrom
fix/readme-pages-badge

Conversation

@tschm

@tschm tschm commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Problem

The weekly link check (run 32705080000) failed with a single lychee error:

[404] https://github.com/tschm/cs/actions/workflows/pages/pages-build-deployment (at 3:1) | Rejected status code: 404 Not Found

Cause

gh api repos/tschm/cs/pages reports "build_type": "workflow" — Pages is deployed by .github/workflows/rhiza_book.yml, not by the classic pages-build-deployment job. That workflow does not exist for this repo, so both the badge image and its link were dead.

Fix

Point the badge at the Book workflow instead. All URLs in README.md were then checked and return 200.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the README book badge to link to the correct build workflow status.

The weekly link check failed because README.md linked to
https://github.com/tschm/cs/actions/workflows/pages/pages-build-deployment,
which 404s. GitHub Pages for this repo uses build_type "workflow"
(deployed by rhiza_book.yml), so the classic pages-build-deployment
workflow does not exist here.

Point the badge at the Book workflow instead.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 95f046ef-6b8f-4117-9234-12e5341ec08a

📥 Commits

Reviewing files that changed from the base of the PR and between cedbbac and 7e8b9fb.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The README book badge now links to and displays the status of the rhiza_book.yml GitHub Actions workflow instead of pages-build-deployment.

Changes

Book badge workflow

Layer / File(s) Summary
Update book badge workflow reference
README.md
The book status badge link and image URL now reference the rhiza_book.yml workflow.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 7e8b9

The README badge now points to the repository's active Book workflow instead of a dead Pages deployment endpoint; this is a localized documentation-only change with no actionable merge-blocking risk remaining.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: replacing the dead pages-build-deployment badge. It is concise and specific.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/readme-pages-badge

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tschm
tschm merged commit b228407 into main Aug 30, 2026
43 checks passed
@tschm
tschm deleted the fix/readme-pages-badge branch August 30, 2026 08:45
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.

1 participant