Skip to content

ci(link-checker): fix root-relative links resolution#6535

Merged
climba03003 merged 3 commits intofastify:mainfrom
barba-rossa:fix-lychee-root-dir
Mar 10, 2026
Merged

ci(link-checker): fix root-relative links resolution#6535
climba03003 merged 3 commits intofastify:mainfrom
barba-rossa:fix-lychee-root-dir

Conversation

@barba-rossa
Copy link
Copy Markdown
Contributor

Fixes #6522

Summary

The link checker was failing to resolve root-relative links such as:

/docs/Reference/TypeScript.md

This happens because lychee requires the --root-dir option to correctly resolve local root-relative paths.

Changes

Added --root-dir . to the lychee arguments in the GitHub Actions workflow.

Result

This fixes false positives from the link checker when running locally and in CI, without affecting runtime code.

@github-actions github-actions bot added the github actions Github actions related label Feb 26, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the internal markdown link-check workflow to correctly resolve root-relative local paths (e.g. /docs/...) by providing Lychee a repository root directory during CI runs.

Changes:

  • Add --root-dir . to Lychee CLI args in the GitHub Actions link-check workflow.

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

You can also share your feedback on Copilot code review. Take the survey.

@climba03003 climba03003 requested review from Eomm, Fdawgs and kibertoad March 8, 2026 00:30
Copy link
Copy Markdown
Member

@Fdawgs Fdawgs left a comment

Choose a reason for hiding this comment

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

Yer a wizard @climba03003

@climba03003 climba03003 merged commit d477915 into fastify:main Mar 10, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github actions Github actions related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Link checker reports ignored

5 participants