Skip to content

fix: fill missing eip4844 settings#4633

Merged
mattsse merged 1 commit intomainfrom
matt/add-missing-fill-settings
Sep 18, 2023
Merged

fix: fill missing eip4844 settings#4633
mattsse merged 1 commit intomainfrom
matt/add-missing-fill-settings

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Sep 18, 2023

configures missing txenv settings for eip-4844

@mattsse mattsse requested review from Rjected and rkrasiuk September 18, 2023 08:35
@mattsse mattsse added the M-eip This change relates to the implementation of an EIP label Sep 18, 2023
@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #4633 (64088c5) into main (a55f48c) will decrease coverage by 0.07%.
Report is 1 commits behind head on main.
The diff coverage is 0.00%.

Impacted file tree graph

Files Changed Coverage Δ
crates/revm/revm-primitives/src/env.rs 40.46% <0.00%> (-0.38%) ⬇️

... and 10 files with indirect coverage changes

Flag Coverage Δ
integration-tests 16.99% <0.00%> (-0.01%) ⬇️
unit-tests 63.16% <0.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
reth binary 31.37% <ø> (ø)
blockchain tree 83.59% <ø> (ø)
pipeline 88.54% <ø> (ø)
storage (db) 73.44% <ø> (ø)
trie 94.70% <ø> (-0.04%) ⬇️
txpool 49.16% <ø> (-0.49%) ⬇️
networking 77.21% <ø> (-0.05%) ⬇️
rpc 57.33% <ø> (-0.13%) ⬇️
consensus 62.42% <ø> (ø)
revm 19.66% <0.00%> (-0.02%) ⬇️
payload builder 9.06% <ø> (ø)
primitives 86.45% <ø> (-0.01%) ⬇️

@mattsse mattsse added this pull request to the merge queue Sep 18, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Sep 18, 2023
@mattsse mattsse merged commit aeb37aa into main Sep 18, 2023
@mattsse mattsse deleted the matt/add-missing-fill-settings branch September 18, 2023 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

M-eip This change relates to the implementation of an EIP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants