Add checklinks to CI - Githubissues

wcarhart / docs

Documentation site for wcarhart's repositories

https://willcarhart.dev/

0 stars 0 forks source link

Add checklinks to CI #7

Closed wcarhart closed 4 years ago

wcarhart commented 4 years ago

checklinks.js currently fails with too many HTTP 429 errors. This needs to be fixed and then node checklinks.js should be added to the CI. Will need to install >= node 12.9 in the action too.

wcarhart commented 4 years ago

To do:

[x] send HTTP request
[x] use DNS for domain resolution
[ ] implement exponential backoff when receive 429
[ ] figure out what to do about Docsify returning 200 instead of 4xx
[ ] add to CI

wcarhart commented 4 years ago

Decided not to do this because of checklinks.js flakiness. Sometimes it will return many 429's due to the large amount of requests.