Skip to content

chore: change field order#4348

Merged
mattsse merged 1 commit intomainfrom
matt/change-accountdiff-order
Aug 24, 2023
Merged

chore: change field order#4348
mattsse merged 1 commit intomainfrom
matt/change-accountdiff-order

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Aug 24, 2023

makes it easier to compare output

@mattsse mattsse requested a review from Rjected as a code owner August 24, 2023 14:14
@mattsse mattsse added the A-rpc Related to the RPC implementation label Aug 24, 2023
@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Merging #4348 (ed7f29a) into main (c2436a9) will decrease coverage by 0.04%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Impacted file tree graph

Files Changed Coverage Δ
crates/rpc/rpc-types/src/eth/trace/parity.rs 66.31% <ø> (ø)

... and 10 files with indirect coverage changes

Flag Coverage Δ
integration-tests 16.71% <ø> (-0.01%) ⬇️
unit-tests 63.72% <ø> (-0.04%) ⬇️

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

Components Coverage Δ
reth binary 26.03% <ø> (ø)
blockchain tree 82.56% <ø> (ø)
pipeline 90.07% <ø> (ø)
storage (db) 74.71% <ø> (ø)
trie 94.88% <ø> (ø)
txpool 47.94% <ø> (-0.53%) ⬇️
networking 77.46% <ø> (-0.08%) ⬇️
rpc 58.04% <ø> (+0.06%) ⬆️
consensus 63.53% <ø> (ø)
revm 31.97% <ø> (ø)
payload builder 6.78% <ø> (ø)
primitives 86.33% <ø> (-0.04%) ⬇️

@mattsse mattsse merged commit 20ea9c9 into main Aug 24, 2023
@mattsse mattsse deleted the matt/change-accountdiff-order branch August 24, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-rpc Related to the RPC implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant