Skip to content
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

Mark HTTP 429 responses as allowed failures in hyperlink check #4253

Merged
merged 1 commit into from
Apr 26, 2020

Conversation

Munter
Copy link
Member

@Munter Munter commented Apr 25, 2020

We sometimes get HTTP 429 responses from unpkg.com when checking the links to mocha and chai assets we use in a cope/pasteable code reference for how to run mocha in a browser.

These checks should still run, but if we know that we might get HTTP 429 because unpkg is temporarily overloaded we should not fail the build.

Example of such a failure: https://app.netlify.com/sites/mocha/deploys/5ea2e0db7751000007185cf1

Adding a --todo match on the error we get in hyperlink allows that particular response to fail without failing the build

…metimes get these from unpkg and it shouldnt block out builds
@Munter Munter added type: chore generally involving deps, tooling, configuration, etc. area: website involving mochajs.org, but not necessarily involving docs labels Apr 25, 2020
@Munter Munter merged commit e7add63 into master Apr 26, 2020
@Munter Munter deleted the munter/handle-unpkg-http-429 branch April 26, 2020 10:35
boneskull pushed a commit that referenced this pull request Apr 29, 2020
…metimes get these from unpkg and it shouldnt block out builds (#4253)
@craigtaub craigtaub added this to the next milestone May 21, 2020
@craigtaub craigtaub added the semver-patch implementation requires increase of "patch" version number; "bug fixes" label May 21, 2020
craigtaub pushed a commit that referenced this pull request May 21, 2020
…metimes get these from unpkg and it shouldnt block out builds (#4253)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: website involving mochajs.org, but not necessarily involving docs semver-patch implementation requires increase of "patch" version number; "bug fixes" type: chore generally involving deps, tooling, configuration, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants