Skip to content

feat(): add relative link validation and update cross-topic references#355

Open
dobromirts wants to merge 16 commits into
vnextfrom
dtsvetkov/add-relative-links-validation
Open

feat(): add relative link validation and update cross-topic references#355
dobromirts wants to merge 16 commits into
vnextfrom
dtsvetkov/add-relative-links-validation

Conversation

@dobromirts

Copy link
Copy Markdown
Contributor

Closes #330, #331

  1. Added check-relative-links:ci npm script and updated the lint CI workflow to validate all relative cross-topic links at build time -generates angular and xplat content first, then checks both trees so no links to generated pages are missed
  2. Extended the checker to accept bare page.mdx links alongside ./page.mdx, excluded _shared/ template directories from direct scanning (links validated via generated output), and added docs/xplat/generated/ to the xplat scan path
  3. Updated README, CONTRIBUTING, and PR template to document the link convention and pipeline

@dobromirts dobromirts marked this pull request as ready for review June 26, 2026 07:23
@dobromirts dobromirts requested a review from viktorkombov June 26, 2026 07:24
@ChronosSF ChronosSF requested a review from Copilot June 29, 2026 15:36

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.

@ChronosSF ChronosSF added the 💥 status: in-test PRs currently being tested label Jun 29, 2026
@ChronosSF

Copy link
Copy Markdown
Member

@dobromirts , you'll have to resolve some conflicts while this is being tested ;)

PR looks good to me on the basis of a quick go-through the files. I'll test how the checker works now.

Comment thread .github/workflows/lint.yml Outdated
Comment thread .github/workflows/lint.yml Outdated
@dobromirts

Copy link
Copy Markdown
Contributor Author

latest reports visible here - reports/relative-links-report.md and reports/html-links-report.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💥 status: in-test PRs currently being tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add cross-topic validation for relative paths at build time

3 participants