Skip to content

Backport release/v6.5: fix(evmrpc): return empty array instead of null for eth_getFilterLogs and eth_getFilterChanges#3325

Merged
masih merged 4 commits intorelease/v6.5from
backport-3292-to-release/v6.5
Apr 28, 2026
Merged

Backport release/v6.5: fix(evmrpc): return empty array instead of null for eth_getFilterLogs and eth_getFilterChanges#3325
masih merged 4 commits intorelease/v6.5from
backport-3292-to-release/v6.5

Conversation

@seidroid
Copy link
Copy Markdown

@seidroid seidroid Bot commented Apr 28, 2026

Backport of #3292 to release/v6.5.

@seidroid
Copy link
Copy Markdown
Author

seidroid Bot commented Apr 28, 2026

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-3292-to-release/v6.5
git worktree add --checkout .worktree/backport-3292-to-release/v6.5 backport-3292-to-release/v6.5
cd .worktree/backport-3292-to-release/v6.5
git reset --hard HEAD^
git cherry-pick -x 0ad5733844536ef11c057e5624bb3e949c99cb9a
git push --force-with-lease

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedApr 28, 2026, 6:11 PM

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

❌ Patch coverage is 25.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.34%. Comparing base (afda866) to head (fe0e28e).
⚠️ Report is 1 commits behind head on release/v6.5.

Files with missing lines Patch % Lines
evmrpc/filter.go 25.00% 4 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##           release/v6.5    #3325   +/-   ##
=============================================
  Coverage         58.34%   58.34%           
=============================================
  Files              2091     2091           
  Lines            173036   173041    +5     
=============================================
+ Hits             100951   100956    +5     
  Misses            63080    63080           
  Partials           9005     9005           
Flag Coverage Δ
sei-chain-pr 63.32% <25.00%> (?)
sei-db 70.41% <ø> (ø)

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

Files with missing lines Coverage Δ
evmrpc/filter.go 69.76% <25.00%> (+0.22%) ⬆️
🚀 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.

@masih masih marked this pull request as ready for review April 28, 2026 19:57
@masih masih merged commit dd76fa4 into release/v6.5 Apr 28, 2026
39 checks passed
@masih masih deleted the backport-3292-to-release/v6.5 branch April 28, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants