-
Notifications
You must be signed in to change notification settings - Fork 235
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
Add GH action to find broken links in Documentation (and elsewhere?) #3342
Comments
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as outdated.
This comment was marked as outdated.
The links in the original bug report post are broken, but it seems like a lot (prob. not all!) of this has been fixed. To put in a test that finds broken links, we could add a GH Action like https://github.com/marketplace/actions/lychee-broken-link-checker to check for broken links on build; it isn't clear to me where to put this in the build process. |
I support automated link checking, with the caveat that any "hit a bunch of external links" task is going to be slower than we think and will see lots of transient failures even from basically-working sites. I vote we put this outside the build process, running on a set schedule (weekly?) rather than every PR and opening issues for (repeated?) failures rather than breaking the build. |
Would this help : #3269 ? |
To put in a test that finds broken links, we could add a GH Action like https://github.com/marketplace/actions/lychee-broken-link-checker to check for broken links on build; it isn't clear to me where to put this in the build process.
Original bug report:
I've noticed that most/all of the links in the documentation that are shortcuts to other parts of the documentation tend to throw a 404 error. This doesn't seem to be true of links that are to external places.
Example
Documentation: "Adding-an-Input-Converter" redirects to this.
Next Steps
The text was updated successfully, but these errors were encountered: