Skip to content

Conversation

@steve-calvert-glean
Copy link
Contributor

This pull request introduces a new GitHub Actions workflow for automated link checking, updates the scripts/check-links.sh script to support configurable checks, and adds a new npm script to simplify local link verification. These changes aim to improve the reliability of documentation links and streamline link-checking processes.

GitHub Actions Workflow for Link Checking:

  • Added a new workflow in .github/workflows/link-checker.yml to automate link checking. It supports both scheduled and manual runs, with configurable options for "quick" or "deep" checks and a customizable base URL. The workflow uses the lychee tool to validate links and creates GitHub issues for broken links detected during scheduled runs.

Enhancements to Link-Checking Script:

  • Updated scripts/check-links.sh to accept a base URL and a "deep check" flag as arguments, enabling more flexible usage. The script now builds a dynamic lychee command with exclusions for known problematic URLs, localhost, and external API endpoints. It also supports deep content crawling when the "deep check" flag is enabled. [1] [2]

New NPM Script for Local Link Checking:

  • Added a new npm script links:check in package.json to run the link-checking script locally with a deep check on the default base URL (https://developers.glean.com).

@steve-calvert-glean steve-calvert-glean requested a review from a team as a code owner July 14, 2025 19:47
@vercel
Copy link

vercel bot commented Jul 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
glean-developer-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2025 7:49pm

@steve-calvert-glean steve-calvert-glean changed the title chore: Adds link checker and associated automation chore: Adds dead link checker and associated automation Jul 14, 2025
@steve-calvert-glean
Copy link
Contributor Author

We should hold off on merging this until we've fixed the outstanding broken links.

🔍 14997 Total (in 30s) ✅ 13666 OK 🚫 14 Errors 👻 1317 Excluded

@rwjblue-glean
Copy link
Member

@steve-calvert-glean - Still want to do this? Should one of us pick up and try to resolve those broken links?

@steve-calvert-glean
Copy link
Contributor Author

Yes. This was just waiting to fix all those broken links, but we should for sure land this.

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.

4 participants