Skip to content

fix: ignore draft docs in link checker#751

Closed
notgiorgi wants to merge 1 commit into
mainfrom
gio/fix-draft-link-check
Closed

fix: ignore draft docs in link checker#751
notgiorgi wants to merge 1 commit into
mainfrom
gio/fix-draft-link-check

Conversation

@notgiorgi
Copy link
Copy Markdown
Contributor

@notgiorgi notgiorgi commented Jun 3, 2026

Summary

  • Add a minimal pnpm project with gray-matter for frontmatter parsing
  • Use the same pnpm SFW settings as ../lightdash: exact save mode plus a 3-day/minimum-release-age install cooldown
  • Skip MDX/Markdown files whose frontmatter title contains draft when running the link checker
  • Mark the timezone draft title as a draft so it is ignored by the checker
  • Install pnpm dependencies in the docs validation workflow before running Node scripts

Testing

  • pnpm install
  • pnpm install --frozen-lockfile
  • node scripts/check-links.js

Tracking

  • No Linear ticket provided

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

🤖 Documentation Bot

All checks passed! No issues found.

@notgiorgi notgiorgi marked this pull request as ready for review June 3, 2026 14:03
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Jun 3, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
lightdash 🟢 Ready View Preview Jun 3, 2026, 2:07 PM

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Jun 3, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgray-matter@​4.0.39910010075100

View full report

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

🤖 Documentation Bot

All checks passed! No issues found.

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.

1 participant