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
replace custom throttling code with @octokit/plugin-throttling #299
Labels
Comments
I'm going to draft a PR on this but I will need some guidance since it's my first contribution to |
achingbrain
added a commit
to achingbrain/github
that referenced
this issue
Apr 8, 2022
Refactors the client to use [@octokit/plugin-retry](https://www.npmjs.com/package/@octokit/plugin-retry) and [@octokit/plugin-throttling](https://www.npmjs.com/package/@octokit/plugin-throttling) as GitHub occasionally changes it's API and these plugins can abstract those changes away from this module. - Removes `lib/definitions/rate-limit.js` - Adds `lib/definitions/retry.js` and `lib/definitions/throttle.js` to handle retry/throttle settings - Updates tests to be more like GitHub (returing the correct rate limit response headers, etc) Fixes semantic-release#299 and semantic-release/semantic-release#2204 See also semantic-release#480 and semantic-release#378
🎉 This issue has been resolved in version 8.0.8 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
as part of that, enable testing in Mac OS and Windows which currently fail because of flaky tests of the throttling code:
https://github.com/semantic-release/github/pull/298/checks?check_run_id=1397908452
The text was updated successfully, but these errors were encountered: