Skip to content

impr(CLDSRV-854): Update rate limit precheck#6130

Open
tmacro wants to merge 1 commit intoimprovement/CLDSRV-855/update_rate_limit_postcheckfrom
improvement/CLDSRV-854/update_rate_limit_precheck
Open

impr(CLDSRV-854): Update rate limit precheck#6130
tmacro wants to merge 1 commit intoimprovement/CLDSRV-855/update_rate_limit_postcheckfrom
improvement/CLDSRV-854/update_rate_limit_precheck

Conversation

@tmacro
Copy link
Copy Markdown
Contributor

@tmacro tmacro commented Apr 1, 2026

No description provided.

@claude
Copy link
Copy Markdown

claude bot commented Apr 1, 2026

LGTM

Review by Claude Code

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 88.28829% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.20%. Comparing base (db880cf) to head (130f937).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/api/api.js 88.28% 13 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/api/api.js 90.53% <88.28%> (+0.61%) ⬆️

... and 8 files with indirect coverage changes

@@                                  Coverage Diff                                   @@
##           improvement/CLDSRV-855/update_rate_limit_postcheck    #6130      +/-   ##
======================================================================================
+ Coverage                                               82.79%   84.20%   +1.41%     
======================================================================================
  Files                                                     204      204              
  Lines                                                   13101    13106       +5     
======================================================================================
+ Hits                                                    10847    11036     +189     
+ Misses                                                   2254     2070     -184     
Flag Coverage Δ
file-ft-tests 67.99% <87.38%> (+0.25%) ⬆️
kmip-ft-tests 28.48% <64.86%> (+9.48%) ⬆️
mongo-v0-ft-tests 69.22% <87.38%> (+0.19%) ⬆️
mongo-v1-ft-tests 69.23% <87.38%> (+0.24%) ⬆️
multiple-backend 35.60% <72.07%> (+16.49%) ⬆️
sur-tests 36.11% <72.07%> (-0.53%) ⬇️
sur-tests-inflights 37.97% <72.07%> (+0.27%) ⬆️
unit 69.66% <63.96%> (-0.02%) ⬇️
utapi-v2-tests 34.81% <72.97%> (+0.24%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants