Skip to content

chore: reorder receipt fields#4291

Merged
onbjerg merged 1 commit intomainfrom
matt/reorder-fields3
Aug 21, 2023
Merged

chore: reorder receipt fields#4291
onbjerg merged 1 commit intomainfrom
matt/reorder-fields3

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Aug 21, 2023

reorder fields to make output easier to compare

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

codecov bot commented Aug 21, 2023

Codecov Report

Merging #4291 (919456d) into main (7f9116b) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

Files Changed Coverage Δ
...rates/rpc/rpc-types/src/eth/transaction/receipt.rs 0.00% <ø> (ø)

... and 11 files with indirect coverage changes

Flag Coverage Δ
integration-tests 16.89% <ø> (+<0.01%) ⬆️
unit-tests 63.73% <ø> (-0.05%) ⬇️

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 48.58% <ø> (-0.53%) ⬇️
networking 77.48% <ø> (+0.03%) ⬆️
rpc 58.70% <ø> (-0.06%) ⬇️
consensus 63.53% <ø> (ø)
revm 32.03% <ø> (ø)
payload builder 6.82% <ø> (ø)
primitives 85.93% <ø> (-0.02%) ⬇️

@onbjerg onbjerg added this pull request to the merge queue Aug 21, 2023
Merged via the queue into main with commit b13424a Aug 21, 2023
@onbjerg onbjerg deleted the matt/reorder-fields3 branch August 21, 2023 12:44
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.

2 participants