Skip to content
This repository has been archived by the owner on Jun 9, 2024. It is now read-only.

feat(cosmos): Use Precommiter hook to store end of block state #540

Merged
merged 15 commits into from
Apr 5, 2023

Conversation

calbera
Copy link
Contributor

@calbera calbera commented Apr 4, 2023

No description provided.

@calbera calbera requested a review from itsdevbear April 4, 2023 23:57
@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

Merging #540 (45a20ef) into main (f79f530) will increase coverage by 0.19%.
The diff coverage is 16.66%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #540      +/-   ##
==========================================
+ Coverage   63.47%   63.67%   +0.19%     
==========================================
  Files          93       94       +1     
  Lines        3762     3755       -7     
==========================================
+ Hits         2388     2391       +3     
+ Misses       1290     1277      -13     
- Partials       84       87       +3     
Impacted Files Coverage Δ
cosmos/lib/bank.go 0.00% <0.00%> (ø)
cosmos/runtime/config/config.go 38.09% <0.00%> (-4.02%) ⬇️
cosmos/x/evm/keeper/keeper.go 87.75% <ø> (ø)
cosmos/x/evm/plugins/block/plugin.go 0.00% <0.00%> (ø)
eth/core/chain_resources.go 0.00% <0.00%> (ø)
eth/core/chain_writer.go 0.00% <0.00%> (ø)
cosmos/x/evm/plugins/gas/plugin.go 72.09% <42.85%> (-16.80%) ⬇️
cosmos/x/evm/keeper/processor.go 63.63% <50.00%> (-3.94%) ⬇️
eth/core/processor.go 48.00% <77.77%> (+1.79%) ⬆️
cosmos/cmd/polard/cmd/root.go 75.12% <100.00%> (+0.24%) ⬆️
... and 1 more

@itsdevbear
Copy link
Contributor

utACK, make an issue to move back off our fork once cosmos/cosmos-sdk#14860 is merged

@itsdevbear itsdevbear merged commit c2955d6 into main Apr 5, 2023
@itsdevbear itsdevbear deleted the cosmos-precommit branch April 5, 2023 18:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants