Skip to content

Conversation

MalteHerrmann
Copy link
Contributor

Description

This PR pulls the latest changes from main and updates the changelog and constants for the v12.0.0-rc2 release.

MalteHerrmann and others added 12 commits March 13, 2023 12:45
* fix(evm): add gas used

* add changelog entry

* add changelog entry

* add changelog entry
* add TxFeeChecker requirement

* add unit tests

* update Changelog
* update CI workflows to use Go 1.20

* update Dockerfile to use Go 1.20

* address linter
…1471)

fix used Go version in CI by passing version as a string
* imp(ante): error message

* add changelog entry

* update err msg

* update err msg to be general for all clients

---------

Co-authored-by: Freddy Caceres <[email protected]>
Co-authored-by: MalteHerrmann <[email protected]>
* deps: update cosmos-sdk version

* add changelog entry

---------

Co-authored-by: MalteHerrmann <[email protected]>
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #1474 (26a3578) into release/v12.0.x (fc0b8e0) will increase coverage by 0.00%.
The diff coverage is 82.35%.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v12.0.x    #1474   +/-   ##
================================================
  Coverage            73.07%   73.08%           
================================================
  Files                  268      268           
  Lines                18445    18446    +1     
================================================
+ Hits                 13479    13481    +2     
- Misses                4364     4365    +1     
+ Partials               602      600    -2     
Impacted Files Coverage Δ
app/ante/cosmos/min_price.go 89.58% <0.00%> (-10.42%) ⬇️
app/ante/evm/fee_checker.go 100.00% <100.00%> (+3.06%) ⬆️
app/ante/evm/fees.go 74.72% <100.00%> (+1.52%) ⬆️
app/ante/handler_options.go 54.28% <100.00%> (+1.34%) ⬆️
rpc/backend/tx_info.go 64.53% <100.00%> (+1.04%) ⬆️

Copy link
Contributor

@GAtom22 GAtom22 left a comment

Choose a reason for hiding this comment

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

LGTM! Great work @MalteHerrmann !!

@MalteHerrmann MalteHerrmann merged commit 215b22c into release/v12.0.x Mar 15, 2023
@MalteHerrmann MalteHerrmann deleted the malte/release-v12.0.0-rc2 branch March 15, 2023 20:56
@austinchandra austinchandra mentioned this pull request Mar 24, 2023
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