Skip to content

fix(ibc): use cache context in ibc timeout callback gas accounting#1063

Open
mmsqe wants to merge 1 commit intocosmos:mainfrom
mmsqe:fix_ctx
Open

fix(ibc): use cache context in ibc timeout callback gas accounting#1063
mmsqe wants to merge 1 commit intocosmos:mainfrom
mmsqe:fix_ctx

Conversation

@mmsqe
Copy link
Copy Markdown
Collaborator

@mmsqe mmsqe commented Mar 10, 2026

Description

Closes: #XXXX


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 10, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 65.48%. Comparing base (bea1ab9) to head (46b7aab).

Files with missing lines Patch % Lines
x/ibc/callbacks/keeper/keeper.go 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1063      +/-   ##
==========================================
+ Coverage   64.11%   65.48%   +1.36%     
==========================================
  Files         331      331              
  Lines       23303    23252      -51     
==========================================
+ Hits        14941    15226     +285     
+ Misses       6946     6884      -62     
+ Partials     1416     1142     -274     
Files with missing lines Coverage Δ
x/ibc/callbacks/keeper/keeper.go 59.15% <0.00%> (ø)

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

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.

1 participant