Skip to content

fix api rate limit errors#18

Merged
Mattraks merged 5 commits intoMattraks:mainfrom
rasa:rasa-fix-api-rate-limit-errors
Dec 18, 2023
Merged

fix api rate limit errors#18
Mattraks merged 5 commits intoMattraks:mainfrom
rasa:rasa-fix-api-rate-limit-errors

Conversation

@rasa
Copy link
Copy Markdown
Contributor

@rasa rasa commented Dec 15, 2023

With a GitHub account that has an active paid subscription, I recently got

API rate limit exceeded for installation ID 1234567.
If you reach out to GitHub Support for help, please include the request ID DEAD:BEEF:DEADED:BEEFFFF:DEADBEEF

in this run.

With node16, this run failed with:

fetch is not set. Please pass a fetch implementation as new Octokit({ request: { fetch }}).
Learn more at https://github.com/octokit/octokit.js/#fetch-missing

https://github.com/octokit/octokit.js/#fetch-missing said to bump to at least node18.

So, after bumping to node20, this run finished successfully, though it doesn't appear that an API rate limit was hit.

@rasa
Copy link
Copy Markdown
Contributor Author

rasa commented Dec 17, 2023

@Mattraks I noticed that this action skips over logs where the action was deleted or renamed. Is that by design, a limitation of the API, or an issue that can be fixed?

@Mattraks Mattraks merged commit 39f0bbe into Mattraks:main Dec 18, 2023
@rasa rasa deleted the rasa-fix-api-rate-limit-errors branch December 18, 2023 15:35
@rasa
Copy link
Copy Markdown
Contributor Author

rasa commented Dec 18, 2023

thanks @Mattraks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants