Skip to content

Move external link checks to a weekly scheduled run#1344

Merged
tomthorogood merged 1 commit into
mainfrom
tomthorogood/weekly-external-link-checks
Jul 13, 2026
Merged

Move external link checks to a weekly scheduled run#1344
tomthorogood merged 1 commit into
mainfrom
tomthorogood/weekly-external-link-checks

Conversation

@tomthorogood

Copy link
Copy Markdown
Contributor

External URL checks are slow and frequently fail in CI due to rate limiting and transient network errors on third-party sites, which can block otherwise valid contributions.

This mirrors the approach taken in github/opensource.guide#3703.

Changes

  • script/html-proofer — External link checking is now opt-in via CHECK_EXTERNAL_LINKS env var (defaults to off)
  • script/test — Explicitly disables external checks so CI stays fast and reliable
  • .github/workflows/link-check.yml — New weekly scheduled workflow (Mondays 08:00 UTC) that runs the full external link check, with workflow_dispatch for manual triggers

External URL checks are slow and frequently fail in CI due to rate
limiting and transient network errors on third-party sites, which can
block otherwise valid contributions.

- Make external link checking opt-in via CHECK_EXTERNAL_LINKS env var
- Disable external checks in script/test (used by CI)
- Add weekly scheduled workflow for full external link validation

Mirrors the approach taken in github/opensource.guide#3703.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 13, 2026 16:51
@tomthorogood tomthorogood requested a review from a team as a code owner July 13, 2026 16:51

Copilot AI 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.

Pull request overview

Moves unreliable external URL validation out of regular CI into a weekly workflow.

Changes:

  • Makes external checks opt-in via CHECK_EXTERNAL_LINKS.
  • Disables external checks in standard tests.
  • Adds weekly and manually triggered full link checking.
Show a summary per file
File Description
script/test Disables external URL checks in normal CI.
script/html-proofer Configures external checking from an environment variable.
.github/workflows/link-check.yml Runs full link validation every Monday or manually.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Medium

@tomthorogood tomthorogood marked this pull request as draft July 13, 2026 16:56
@tomthorogood tomthorogood marked this pull request as ready for review July 13, 2026 16:59
@tomthorogood tomthorogood merged commit 8e93266 into main Jul 13, 2026
6 checks passed
@tomthorogood tomthorogood deleted the tomthorogood/weekly-external-link-checks branch July 13, 2026 17:28
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.

3 participants