fix: rate limit err #261
fix: rate limit err #261
Conversation
|
Does this make use of https://github.com/octokit/plugin-throttling.js/#readme? This is the official GitHub recommended way! Source: https://github.com/octokit/octokit.js?tab=readme-ov-file#request-error-handling |
Cool, I didn't know that. Will have to rewrite some part of the action. Thanks |
|
@koppor would you like to give this a shot? In the meantime, I will try resolving other issues in this pr before merging |
|
The output is
(Link to action output: https://github.com/JabRef/jabref/actions/runs/12296140810/job/34314530086) But I have 23 issues with the label "Assigned": https://github.com/JabRef/jabref/issues?q=sort%3Aupdated-desc%20is%3Aissue%20is%3Aopen%20label%3A%22%F0%9F%93%8D%20Assigned%22 I think, the number should be "23", shouldn't it? Update: I checked the code and it appears that |
|
Did not work 😢 https://github.com/JabRef/jabref/actions/runs/12315417134 You have exceeded a secondary rate limit. Please wait a few minutes before you try again. If you reach out to GitHub Support for help, please include the request ID 2402:3CBDD4:2C54AC9:57ADDC6:675C24AE. - https://docs.github.com/free-pro-team@latest/rest/overview/rate-limits-for-the-rest-api#about-secondary-rate-limits |
|
Sorry for the delay, looking into it |
|
@takanome-dev May I ask if you could find some time to work on it? GSoC is coming up and this action really helps to manage the workload. |
What type of PR is this? (check all applicable)
Description
This pr fixes the issues where users hit the second rate limit
Related Tickets & Documents
fixes #251
Mobile & Desktop Screenshots/Recordings
Added tests?
Added to documentation?
[optional] What gif best describes this PR or how it makes you feel?