Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: release vm right after execution finished #274

Merged
merged 2 commits into from
Sep 26, 2024

fix to charge gas first before returnning errorg

7791444
Select commit
Loading
Failed to load commit list.
Merged

fix: release vm right after execution finished #274

fix to charge gas first before returnning errorg
7791444
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 26, 2024 in 0s

40.64% (-0.03%) compared to a0b130a

View this Pull Request on Codecov

40.64% (-0.03%) compared to a0b130a

Details

Codecov Report

Attention: Patch coverage is 92.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 40.64%. Comparing base (a0b130a) to head (7791444).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
x/move/keeper/handler.go 91.89% 2 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
- Coverage   40.67%   40.64%   -0.03%     
==========================================
  Files         265      265              
  Lines       25258    25246      -12     
==========================================
- Hits        10274    10262      -12     
  Misses      13396    13396              
  Partials     1588     1588
Files with missing lines Coverage Δ
x/move/keeper/genesis.go 16.17% <100.00%> (-1.22%) ⬇️
x/move/keeper/handler.go 78.57% <91.89%> (-0.54%) ⬇️
View more details on Codecov