Skip to content

Conversation

@bbbco
Copy link
Contributor

@bbbco bbbco commented Apr 28, 2025

Fixes #56 by adding handling for 429 responses

@codecov
Copy link

codecov bot commented Apr 30, 2025

Codecov Report

Attention: Patch coverage is 3.22581% with 30 lines in your changes missing coverage. Please review.

Project coverage is 46.29%. Comparing base (137bf38) to head (d8e507d).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
scraper/http.go 0.00% 30 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
- Coverage   47.55%   46.29%   -1.27%     
==========================================
  Files          14       14              
  Lines        1024     1052      +28     
==========================================
  Hits          487      487              
- Misses        485      513      +28     
  Partials       52       52              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cornelk
Copy link
Owner

cornelk commented Apr 30, 2025

thank you for the bug report and fix!

@cornelk cornelk merged commit 1fe4bf9 into cornelk:main Apr 30, 2025
2 of 3 checks passed
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.

Add support for handling retries after 429 status code

2 participants