Skip to content

[docs] Add CI dead link gate - #3923

Open
dataroaring wants to merge 1 commit into
apache:masterfrom
dataroaring:codex/ci-dead-link-blocker
Open

dataroaring wants to merge 1 commit into
apache:masterfrom
dataroaring:codex/ci-dead-link-blocker

Conversation

@dataroaring

Copy link
Copy Markdown
Contributor

Summary

  • make the docs changed-link check fail CI on blocking internal link errors
  • expand the docs governance manifest to cover current docs, zh-CN current docs, and site page routes such as /download
  • fix repo-internal missing targets and anchors across active, archived, localized, and ja-source docs
  • improve link linting for compatibility anchors, dotted relative paths, fenced code masking, and report-only severity handling

Validation

  • node scripts/docs-governance/lint-links.js --output /tmp/doris-links-all-no404-audit.json
    • link-missing-target = 0
    • link-missing-anchor = 0
    • errors = 0
  • yarn docs:links:changed --fail-on-errors --output /tmp/doris-links-changed-no404-audit.json
    • internal_missing = 0
    • errors = 0
  • yarn docs-governance:test
  • git diff --check

Notes

External links remain report-only and are not fetched by this gate; this PR eliminates repo-internal 404 references covered by the docs link linter.

Copilot AI review requested due to automatic review settings June 10, 2026 02:10

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

This branch had an error being deployed

1 failed deployment
Production — 9cfbd932 Deployed Jun 10, 2026 by dataroaring via Build Check #7941
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.

2 participants