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.

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

[EVM] Tune configs #1800

Merged
merged 9 commits into from
Aug 2, 2024
Merged

[EVM] Tune configs #1800

merged 9 commits into from
Aug 2, 2024

Conversation

jewei1997
Copy link
Contributor

Describe your changes and provide context

Increase gas price based on gas price investigation.

Testing performed to validate your change

unit tests. Also performed manual eth transfer on local chain and verified balance change is expected.

@philipsu522 philipsu522 changed the title [EVM] Gas price change [EVM] Tune configs Aug 1, 2024
Copy link

codecov bot commented Aug 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.56%. Comparing base (e8e4b3b) to head (b91b286).
Report is 129 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1800      +/-   ##
==========================================
- Coverage   61.64%   60.56%   -1.09%     
==========================================
  Files         365      380      +15     
  Lines       26178    27956    +1778     
==========================================
+ Hits        16138    16932     +794     
- Misses       8967     9896     +929     
- Partials     1073     1128      +55     
Files Coverage Δ
x/evm/types/params.go 69.11% <ø> (-0.89%) ⬇️

... and 75 files with indirect coverage changes

Copy link
Collaborator

@codchen codchen left a comment

Choose a reason for hiding this comment

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

we'd still need to send a gov proposal to update it for arctic/atlantic/pacific (this PR changes the default which only affects new chains)

@jewei1997 jewei1997 enabled auto-merge (squash) August 2, 2024 20:49
@jewei1997 jewei1997 merged commit 033a8b5 into main Aug 2, 2024
49 checks passed
@jewei1997 jewei1997 deleted the evm-min-gas-price-increase branch August 2, 2024 20:51
yzang2019 added a commit that referenced this pull request Aug 15, 2024
* main:
  Add more DEX dapp tests (#1809)
  Add basic LST integration tests  (#1814)
  Allow CW->ERC pointers to be called through wasmd precompile (#1785)
  Bump nonce even if tx fails (#1778)
  Fix docker setup for local cluster (#1806)
  Tune Configs (#1813)
  V5.7.5 release (#1805)
  Evidence Max Bytes Update (#1812)
  Add dApp Tests (#1802)
  [EVM] Tune configs (#1800)
  Revert dex removal (#1801)
  Do not charge gas for feecollector address query (#1795)
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.

2 participants