Skip to content

Add lychee link checking to CI - #1603

Open
aaazzam wants to merge 1 commit into
mainfrom
link-check-ci
Open

Add lychee link checking to CI#1603
aaazzam wants to merge 1 commit into
mainfrom
link-check-ci

Conversation

@aaazzam

@aaazzam aaazzam commented Jul 6, 2026

Copy link
Copy Markdown

On PRs, checks links only in changed .py/.md/.ipynb files (full sweep if lychee.toml or the workflow itself changed). Twice a week, sweeps everything, opens or updates a broken-links tracking issue with the report, closes it when clean, and refreshes the result cache that PR runs restore. Successes are cached for 10 days; 429s and 5xx are never cached, so transient failures can't stick.

Every exclusion in lychee.toml is there for a reason:

  • { / %7B / ... / your- / my- / APP_ID / example.com / .modal.host — templated and placeholder URLs
  • oauth2 — lychee mis-parses https://oauth2:{token}@github.com/... into a check of the literal host oauth2
  • tensorflow.org — anonymous requests loop forever through a prompt=none OAuth redirect
  • chaiassets.com, github.com/*/releases/download/*/ — base URLs that code appends filenames to

The full sweep currently finds 31 genuinely broken links, so this PR's own check run is red. Fixing those is a separate change; once this merges, the scheduled job will track them in the issue.


Open in Devin Review

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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