Skip to content

Comments

fix: fix flaky test#250

Merged
gakonst merged 1 commit intomainfrom
matt/fix-flaky-test
Nov 24, 2022
Merged

fix: fix flaky test#250
gakonst merged 1 commit intomainfrom
matt/fix-flaky-test

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Nov 24, 2022

await established session on both ends.

@mattsse mattsse added A-devp2p Related to the Ethereum P2P protocol C-test A change that impacts how or what we test labels Nov 24, 2022
@codecov
Copy link

codecov bot commented Nov 24, 2022

Codecov Report

Merging #250 (ed184a5) into main (f4dbedd) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #250      +/-   ##
==========================================
+ Coverage   73.04%   73.07%   +0.03%     
==========================================
  Files         227      227              
  Lines       20516    20516              
==========================================
+ Hits        14985    14992       +7     
+ Misses       5531     5524       -7     
Impacted Files Coverage Δ
crates/stages/src/stages/bodies.rs 95.80% <0.00%> (-0.24%) ⬇️
crates/net/discv4/src/lib.rs 68.59% <0.00%> (+0.29%) ⬆️
crates/tracing/src/lib.rs 100.00% <0.00%> (+85.71%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@gakonst gakonst merged commit 2e3c220 into main Nov 24, 2022
@gakonst gakonst deleted the matt/fix-flaky-test branch November 24, 2022 21:26
emhane added a commit to JoshdfG/reth that referenced this pull request Jan 5, 2026
…gmxyz#250)

Ref op-rs#241

- Fixes comment from deps fix in
op-rs#229 (fix correct but comment
misleading)
- `serde-bincode-compat` is a feature which must be enabled when the
`bincode` dependency is used. Asides for dev-deps, this only happens in
`reth-stages`. `reth-exex` runs in `reth-stages`. In this wokrstream we
use the `TrieUpdate` in exex. `serde-bincode-compat` is already
implemented for `TrieUpdates` in `reth-trie-common`, this PR simply
makes that feature accessible via `reth-trie` and in turn
`reth-optimism-trie`, in order to enable it in `reth-optimism-exex`.
JoshdfG pushed a commit to JoshdfG/reth that referenced this pull request Jan 16, 2026
…gmxyz#250)

Ref op-rs#241

- Fixes comment from deps fix in
op-rs#229 (fix correct but comment
misleading)
- `serde-bincode-compat` is a feature which must be enabled when the
`bincode` dependency is used. Asides for dev-deps, this only happens in
`reth-stages`. `reth-exex` runs in `reth-stages`. In this wokrstream we
use the `TrieUpdate` in exex. `serde-bincode-compat` is already
implemented for `TrieUpdates` in `reth-trie-common`, this PR simply
makes that feature accessible via `reth-trie` and in turn
`reth-optimism-trie`, in order to enable it in `reth-optimism-exex`.
emhane added a commit that referenced this pull request Jan 21, 2026
Ref op-rs#241

- Fixes comment from deps fix in
op-rs#229 (fix correct but comment
misleading)
- `serde-bincode-compat` is a feature which must be enabled when the
`bincode` dependency is used. Asides for dev-deps, this only happens in
`reth-stages`. `reth-exex` runs in `reth-stages`. In this wokrstream we
use the `TrieUpdate` in exex. `serde-bincode-compat` is already
implemented for `TrieUpdates` in `reth-trie-common`, this PR simply
makes that feature accessible via `reth-trie` and in turn
`reth-optimism-trie`, in order to enable it in `reth-optimism-exex`.
emhane added a commit to JoshdfG/reth that referenced this pull request Jan 21, 2026
…gmxyz#250)

Ref op-rs#241

- Fixes comment from deps fix in
op-rs#229 (fix correct but comment
misleading)
- `serde-bincode-compat` is a feature which must be enabled when the
`bincode` dependency is used. Asides for dev-deps, this only happens in
`reth-stages`. `reth-exex` runs in `reth-stages`. In this wokrstream we
use the `TrieUpdate` in exex. `serde-bincode-compat` is already
implemented for `TrieUpdates` in `reth-trie-common`, this PR simply
makes that feature accessible via `reth-trie` and in turn
`reth-optimism-trie`, in order to enable it in `reth-optimism-exex`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-devp2p Related to the Ethereum P2P protocol C-test A change that impacts how or what we test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants