Skip to content

fix double refund#2691

Closed
jewei1997 wants to merge 2 commits intorelease/v6.3from
release6.3/fix-double-refund
Closed

fix double refund#2691
jewei1997 wants to merge 2 commits intorelease/v6.3from
release6.3/fix-double-refund

Conversation

@jewei1997
Copy link
Copy Markdown
Contributor

Describe your changes and provide context

Testing performed to validate your change

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 12, 2026

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

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedJan 12, 2026, 2:59 PM

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.96%. Comparing base (5e91a00) to head (cb799f9).
⚠️ Report is 18 commits behind head on release/v6.3.

❌ Your project status has failed because the head coverage (38.19%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v6.3    #2691      +/-   ##
================================================
- Coverage         43.38%   41.96%   -1.43%     
================================================
  Files              1854     1066     -788     
  Lines            153857    83324   -70533     
================================================
- Hits              66758    34966   -31792     
+ Misses            81137    45358   -35779     
+ Partials           5962     3000    -2962     
Flag Coverage Δ
sei-chain ?
sei-cosmos 37.95% <ø> (-0.01%) ⬇️
sei-db 44.98% <ø> (+0.06%) ⬆️
sei-ibc-go 55.96% <ø> (ø)
sei-tendermint ?
sei-wasmd 42.06% <ø> (-0.02%) ⬇️
sei-wasmvm 39.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 801 files with indirect coverage changes

🚀 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.

./sei-wasmvm
)

replace github.com/ethereum/go-ethereum => github.com/sei-protocol/go-ethereum v1.15.7-sei-9.0.20260112140943-77a0432740be
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of replace directive here, can I ask you to update the actual version in the go mods please?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jewei1997 we no longer need this replace directive now that the actual version in go.mod is updated. Right?

@github-actions
Copy link
Copy Markdown

We truly appreciate your contribution and the time you’ve invested in this PR.
Before we can merge it, we’d love your help addressing the remaining feedback or sharing your perspective.
If we don’t hear back within 2 day(s), this PR will close automatically — but don’t worry, you can reopen it anytime when you’re ready to continue.

@jewei1997
Copy link
Copy Markdown
Contributor Author

merged here #2692

@jewei1997
Copy link
Copy Markdown
Contributor Author

merged to main here
#2692

and to v6.3 here
#2694

@jewei1997 jewei1997 closed this Feb 10, 2026
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.

3 participants