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

feat: fee market module #491

Merged
merged 10 commits into from
Aug 26, 2021
Merged

feat: fee market module #491

merged 10 commits into from
Aug 26, 2021

Conversation

fedekunze
Copy link
Contributor

@fedekunze fedekunze commented Aug 25, 2021

This PR migrates all the EIP1559 into an independent module: x/feemarket

@github-actions github-actions bot added the C:Proto protobuf files (*.pb.go) label Aug 25, 2021
@github-actions github-actions bot added C:CLI Type: Docs Improvements or additions to documentation labels Aug 25, 2021
@codecov
Copy link

codecov bot commented Aug 26, 2021

Codecov Report

Merging #491 (eebab80) into main (4b11ac6) will increase coverage by 1.36%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #491      +/-   ##
==========================================
+ Coverage   47.56%   48.92%   +1.36%     
==========================================
  Files          57       56       -1     
  Lines        5466     5326     -140     
==========================================
+ Hits         2600     2606       +6     
+ Misses       2749     2603     -146     
  Partials      117      117              
Impacted Files Coverage Δ
x/evm/keeper/abci.go 23.52% <ø> (+10.19%) ⬆️
x/evm/keeper/keeper.go 70.68% <ø> (+8.88%) ⬆️
x/evm/types/dynamic_fee_tx.go 0.00% <ø> (ø)
x/evm/types/query.pb.gw.go 0.00% <ø> (ø)
app/app.go 83.80% <100.00%> (+0.34%) ⬆️
x/evm/types/params.go 64.78% <100.00%> (-0.08%) ⬇️

@odeke-em
Copy link
Contributor

@fedekunze please rebase all these PRs with the code from the main/master branch and then they'll have associated benchmarks.

@orijbot
Copy link

orijbot commented Aug 26, 2021

@fedekunze fedekunze marked this pull request as ready for review August 26, 2021 09:54
@fedekunze fedekunze enabled auto-merge (squash) August 26, 2021 09:55
@fedekunze fedekunze merged commit f469db9 into main Aug 26, 2021
@fedekunze fedekunze deleted the fee-market branch August 26, 2021 10:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C:CLI C:Proto protobuf files (*.pb.go) Type: Docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants