Skip to content

fix: use i64 instead of u64 in limit exceed error to avoid conversion#6872

Merged
akaladarshi merged 1 commit into
mainfrom
akaladarshi/fix-limit-exceeded-err
Apr 8, 2026
Merged

fix: use i64 instead of u64 in limit exceed error to avoid conversion#6872
akaladarshi merged 1 commit into
mainfrom
akaladarshi/fix-limit-exceeded-err

use i64 instead of u64 in limit exceed error to avoid conversion

28c608a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 8, 2026 in 0s

64.02% (-0.08%) compared to 19d1045

View this Pull Request on Codecov

64.02% (-0.08%) compared to 19d1045

Details

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.02%. Comparing base (19d1045) to head (28c608a).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/rpc/methods/eth.rs 0.00% 1 Missing ⚠️
src/rpc/methods/eth/filter/mod.rs 66.66% 1 Missing ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
src/rpc/methods/eth/errors.rs 69.38% <100.00%> (ø)
src/rpc/methods/eth.rs 65.46% <0.00%> (ø)
src/rpc/methods/eth/filter/mod.rs 88.33% <66.66%> (ø)

... and 21 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4cfb27b...28c608a. Read the comment docs.

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