Skip to content

Remove duplicate panic recovery in process proposal#2419

Merged
masih merged 1 commit into
mainfrom
masih/cosmos-port-601
Oct 1, 2025
Merged

Remove duplicate panic recovery in process proposal#2419
masih merged 1 commit into
mainfrom
masih/cosmos-port-601

Conversation

@masih
Copy link
Copy Markdown
Collaborator

@masih masih commented Oct 1, 2025

Port of original work in sei-protocol/sei-cosmos#601

The panic recovery block seems to be repeated twice. This might have been an error rebasing or found its way in by accident?

The panic recovery block seems to be repeated twice. This might have
been an error rebasing or found its way in by accident?
@masih masih requested review from pompon0 and sei-will October 1, 2025 09:31
@masih masih enabled auto-merge (squash) October 1, 2025 09:33
@masih masih disabled auto-merge October 1, 2025 09:38
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 36.69%. Comparing base (f4ab4d1) to head (32cb501).
⚠️ Report is 1 commits behind head on main.

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

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #2419       +/-   ##
===========================================
- Coverage   46.72%   36.69%   -10.04%     
===========================================
  Files         408     1165      +757     
  Lines       39574   104343    +64769     
===========================================
+ Hits        18491    38285    +19794     
- Misses      19796    62810    +43014     
- Partials     1287     3248     +1961     
Files with missing lines Coverage Δ
sei-cosmos/baseapp/abci.go 26.38% <ø> (ø)

... and 908 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.

@masih masih enabled auto-merge (squash) October 1, 2025 09:56
@masih masih requested a review from yzang2019 October 1, 2025 10:05
@masih masih merged commit 62d017e into main Oct 1, 2025
80 of 84 checks passed
@masih masih deleted the masih/cosmos-port-601 branch October 1, 2025 10:25
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