-
Notifications
You must be signed in to change notification settings - Fork 0
Fix link check CI configuration error and broken documentation links #78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
|
@copilot link check failed |
…anager reference Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Fixed! The link check was failing due to 7 broken links in the documentation. All issues have been resolved in commit 8c9bfaa: Fixed links:
All links now validate successfully. The CI should pass on the next run. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes CI link check failures caused by invalid lychee configuration and broken documentation links. The changes ensure the link checker runs successfully by updating the configuration to match lychee v0.21.0 requirements and correcting all broken links found during validation.
Changes:
- Fixed
lychee.tomlconfiguration by changingverbosefrom boolean to string log level - Added
.lycheecacheto.gitignoreto exclude lychee's cache directory - Replaced 6 broken GitHub Discussions links with GitHub Issues (discussions not enabled for this repository)
- Fixed 1 broken example reference from non-existent
task-managerto existingtodoexample
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| lychee.toml | Updated verbose parameter from true to "info" to match lychee v0.21.0 string log level requirement |
| .gitignore | Added .lycheecache entry to exclude lychee link checker cache directory |
| content/docs/tutorials.mdx | Replaced 2 GitHub Discussions links with GitHub Issues links |
| content/docs/troubleshooting.mdx | Replaced 1 GitHub Discussions link with GitHub Issues link |
| content/docs/specifications/index.mdx | Replaced 1 GitHub Discussions link with GitHub Issues link |
| content/docs/quick-start/build-first-app.mdx | Fixed broken example reference from non-existent task-manager to existing todo |
| content/docs/faq.mdx | Replaced 1 GitHub Discussions link with GitHub Issues link |
| content/blog/protocol-first-development.mdx | Replaced 1 GitHub Discussions link with GitHub Issues link |
| content/blog/metadata-driven-architecture.mdx | Replaced 1 GitHub Discussions link with GitHub Issues link |
Link check CI was failing due to two issues: invalid configuration in
lychee.tomland broken links in the documentation.Changes
verbose = truetoverbose = "info"(lychee v0.21.0 expects a string log level).lycheecacheto exclude lychee's cache directoryexamples/task-managerreference withexamples/todoConfiguration Fix
Broken Links Fixed
Files updated:
content/blog/metadata-driven-architecture.mdxcontent/blog/protocol-first-development.mdxcontent/docs/specifications/index.mdxcontent/docs/tutorials.mdxcontent/docs/troubleshooting.mdxcontent/docs/faq.mdxcontent/docs/quick-start/build-first-app.mdxAll links now validate successfully with lychee v0.21.0.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.