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

fix(github): check rate limit again after sleeping #4152

Merged
merged 3 commits into from
Jun 29, 2023
Merged

fix(github): check rate limit again after sleeping #4152

merged 3 commits into from
Jun 29, 2023

Conversation

caarlos0
Copy link
Member

also ensure sleep > 0

also ensure sleep > 0

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the bug Something isn't working label Jun 29, 2023
@caarlos0 caarlos0 self-assigned this Jun 29, 2023
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 29, 2023
@cloudflare-pages
Copy link

cloudflare-pages bot commented Jun 29, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: bb8398c
Status: ✅  Deploy successful!
Preview URL: https://dfb5f7cf.goreleaser.pages.dev
Branch Preview URL: https://gh-rate.goreleaser.pages.dev

View logs

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jun 29, 2023
@codecov
Copy link

codecov bot commented Jun 29, 2023

Codecov Report

Merging #4152 (bb8398c) into main (6b46a1a) will decrease coverage by 0.04%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##             main    #4152      +/-   ##
==========================================
- Coverage   84.06%   84.02%   -0.04%     
==========================================
  Files         130      130              
  Lines       12278    12285       +7     
==========================================
+ Hits        10322    10323       +1     
- Misses       1553     1558       +5     
- Partials      403      404       +1     
Impacted Files Coverage Δ
internal/client/github.go 60.04% <14.28%> (-0.75%) ⬇️

@caarlos0 caarlos0 merged commit f883131 into main Jun 29, 2023
10 of 12 checks passed
@caarlos0 caarlos0 deleted the gh-rate branch June 29, 2023 17:00
@github-actions github-actions bot added this to the v1.20.0 milestone Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant