Skip to content

feat: track active forks#4315

Merged
mattsse merged 1 commit intomainfrom
matt/track-forks
Aug 22, 2023
Merged

feat: track active forks#4315
mattsse merged 1 commit intomainfrom
matt/track-forks

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Aug 22, 2023

activate supported forks in the tx validator based on latest block

@mattsse mattsse added M-changelog This change should be included in the changelog A-tx-pool Related to the transaction mempool labels Aug 22, 2023
@codecov
Copy link

codecov bot commented Aug 22, 2023

Codecov Report

Merging #4315 (8c84819) into main (6fee16e) will decrease coverage by 0.08%.
Report is 2 commits behind head on main.
The diff coverage is 0.00%.

Impacted file tree graph

Files Changed Coverage Δ
crates/transaction-pool/src/validate/eth.rs 0.00% <0.00%> (ø)
crates/transaction-pool/src/validate/task.rs 0.00% <0.00%> (ø)

... and 12 files with indirect coverage changes

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

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

Components Coverage Δ
reth binary 26.12% <ø> (ø)
blockchain tree 82.56% <ø> (ø)
pipeline 90.07% <ø> (ø)
storage (db) 74.77% <ø> (ø)
trie 94.85% <ø> (ø)
txpool 47.81% <0.00%> (-0.93%) ⬇️
networking 77.50% <ø> (-0.08%) ⬇️
rpc 58.82% <ø> (+<0.01%) ⬆️
consensus 63.53% <ø> (ø)
revm 32.01% <ø> (ø)
payload builder 6.82% <ø> (ø)
primitives 86.14% <ø> (-0.03%) ⬇️

@mattsse mattsse requested a review from Rjected August 22, 2023 15:21
Copy link
Member

@Rjected Rjected left a comment

Choose a reason for hiding this comment

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

cool, ForkTracker seems like a good way to handle this

@mattsse mattsse added this pull request to the merge queue Aug 22, 2023
Merged via the queue into main with commit 404f6ba Aug 22, 2023
@mattsse mattsse deleted the matt/track-forks branch August 22, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tx-pool Related to the transaction mempool M-changelog This change should be included in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants