-
Notifications
You must be signed in to change notification settings - Fork 8
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
Upgrade link checker to open an issue if there are broken links #15
Comments
Attempt to igore twitter links during checking, for #15.
Just tweaking the config seems to be sufficient. |
Hmm, also looking at digipres/policies#10 (comment) and I think a more sophisticated solution is justified. The main reason for this is that workflow error notifications only go to whoever touched the workflow last, which is not really appropriate in this case. What we really want is a link checking approach we can use across multiple repos and that will record the results as a GitHub issue. After spending some time exploring the different GitHub Actions currently available, this seems to be a really good approach: https://github.com/marketplace/actions/lychee-broken-link-checker It can to HTML as well as Markdown, and is more configurable, e.g. caching and setting retry-delays (which have caused problems here, forcing me to switch-off checks for some URLs because the current process was retrying too fast and couldn't be slowed down). The only problem seems to be implemented as-is, it will add a new issue every time it runs, even if there's already an open issue. However, it may be possible to combine it with https://github.com/JasonEtco/create-an-issue to find an existing issue and pass the |
Another transient false-positive today log. Presumably the DOI resolver didn't like us going too fast. |
Another Lychee GitHub Action here: https://github.com/lycheeverse/lychee-action |
See also: |
The linting workflow includes a link checker, but this is going off all the time and it's not helpful.
These links are not really dead. It's just Twitter blocking things.
The text was updated successfully, but these errors were encountered: