Skip to content

chore: add TooManyRequestsError to http error#4311

Merged
czy88840616 merged 1 commit intomidwayjs:mainfrom
liuyuan512:add_too_many_requests_error
Apr 10, 2025
Merged

chore: add TooManyRequestsError to http error#4311
czy88840616 merged 1 commit intomidwayjs:mainfrom
liuyuan512:add_too_many_requests_error

Conversation

@liuyuan512
Copy link
Copy Markdown
Contributor

@liuyuan512 liuyuan512 commented Mar 21, 2025

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

@waitingsong
Copy link
Copy Markdown
Member

429?

@czy88840616 czy88840616 added the pr: new feature This PR adds a new API or behavior. label Mar 22, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 22, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.20%. Comparing base (a3ca53b) to head (82b91c4).
Report is 592 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4311      +/-   ##
==========================================
+ Coverage   84.55%   85.20%   +0.65%     
==========================================
  Files         491      528      +37     
  Lines       46778    50780    +4002     
  Branches     5601     5114     -487     
==========================================
+ Hits        39551    43269    +3718     
- Misses       7191     7488     +297     
+ Partials       36       23      -13     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@czy88840616 czy88840616 added pr: polish This PR adds a very minor behavior improvement that users will enjoy. and removed pr: new feature This PR adds a new API or behavior. labels Apr 10, 2025
@czy88840616 czy88840616 merged commit cd5eaef into midwayjs:main Apr 10, 2025
7 checks passed
czy88840616 pushed a commit that referenced this pull request Jun 19, 2025
Co-authored-by: shishang.ly <shishang.ly@alibaba-inc.com>
(cherry picked from commit cd5eaef)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: polish This PR adds a very minor behavior improvement that users will enjoy.

Development

Successfully merging this pull request may close these issues.

4 participants