-
Notifications
You must be signed in to change notification settings - Fork 375
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
Expose rate limits on the logs #1029
Comments
Hello @IOrlandoni , thank you for you suggestion. I looks very useful and will be implemented asap |
Hey @dsame, just wanted to confirm if you're working on it. If you are, I wont :) |
Hi @IOrlandoni , i am already close to providing the solution, but i'd like to see you approach because this statement "The information should already be included in the headers of the responses from Github" is not clear for me. According to the docs the response does not include api usage info: see response schema |
Hi @IOrlandoni, debug status before the processing: https://github.com/akv-demo/test-stale/actions/runs/5257429320/jobs/9500195867#step:2:40 debug status after the processing: https://github.com/akv-demo/test-stale/actions/runs/5257429320/jobs/9500195867#step:2:208 and summary of the usage logged on info level: |
@dsame LGTM. Can't complain! Thank you! |
Hello @IOrlandoni , the related PR is merged and will be available in the next release. Meanwhile you can test and review the change by switching to the main branch
|
Description:
It would be nice to get an idea of the rate limit information before or after the run, to understand the context on which the action is running. The information should already be included in the headers of the responses from Github, it's just a matter of displaying it.
Justification:
Understanding if we can raise the operations per run, if we're close to hitting the rate limit, etc.
Are you willing to submit a PR?
Happy to, if someone confirms this is a good thing for the action.
The text was updated successfully, but these errors were encountered: