Skip to content

ci(docs): 🔗 add source and post-build docs link checks - #53

Draft
matteoacrossi wants to merge 5 commits into
mainfrom
matteoacrossi-docs-link-checking
Draft

ci(docs): 🔗 add source and post-build docs link checks#53
matteoacrossi wants to merge 5 commits into
mainfrom
matteoacrossi-docs-link-checking

Conversation

@matteoacrossi

Copy link
Copy Markdown
Member

Why

The docs pipeline did not enforce broken-link checks, so invalid links could slip through. We need link validation both before build (fast feedback on docs sources) and after build (final exported HTML correctness, including external URLs).

What changed

  • Added a source-level check with lychee for docs MDX/Markdown:
    • new .lychee.toml (offline/internal-focused settings)
    • new just check-doc-links recipe
  • Added a post-build check for exported site HTML:
    • new .lychee.postbuild.toml (external links enabled, bounded retries/timeouts)
    • new just check-doc-links-built recipe (runs after build-docs)
  • Updated docpages.yml to run both checks in CI:
    1. source docs links
    2. built docs links (including external)
  • Updated README.md with the new docs link-check commands.

Notes for reviewers

  • The source check uses a quoted recursive glob so top-level docs files are included reliably.
  • Post-build checking intentionally allows HTTP 429 as acceptable to reduce CI flakiness from rate limits.

matteoacrossi and others added 4 commits July 9, 2026 17:19
Add lychee-based docs link checking for both source MDX (internal links) and built HTML output (including external links), and wire both checks into the documentation workflow. Update docs commands in README accordingly.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com>
@robertodr

Copy link
Copy Markdown
Member

@matteoacrossi are you still interested in getting this merged?

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