Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

ci(automation): add Linkinator GitHub Action #658

Merged
merged 10 commits into from
Mar 11, 2021
Merged

Conversation

bnb
Copy link
Contributor

@bnb bnb commented Jan 12, 2021

This PR adds Linkinator's GitHub Action. Theoretically as a GitHub Action, it'll less aggressively hit the GitHub rate limit which we trigger pretty fast. Not hopeful that it'll work on the first try, but hopeful that we can at least eventually get there with Linkinator... especially since it's by @JustinBeckwith who is wonderful.

@JustinBeckwith
Copy link

JustinBeckwith commented Jan 12, 2021

If you're running into issues with the GitHub rate limit, you can also add a retry option set to true now that will automatically respect the retry-after HTTP header when GitHub returns a HTTP 429. I just pulled this up from the linkinator CLI to the action so you can use it 😄 Only downside here is that the retry-after from GitHub is typically set to 60 seconds, so it can delay your build a bit if it's triggered.

Base automatically changed from master to main January 15, 2021 19:51
@bnb
Copy link
Contributor Author

bnb commented Mar 4, 2021

@nodejs/community-committee this should be ready to go. The only link that's failing still is one that's fixed in #667.

Thank you @JustinBeckwith for your help ❤️

@JustinBeckwith
Copy link

Absolutely! Please if you have any questions at all, I'm always happy to hop in and help.

Copy link
Member

@joesepi joesepi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@bnb bnb merged commit 205d386 into main Mar 11, 2021
@bnb bnb deleted the bnb/linkinator-action branch March 11, 2021 18:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants