Skip to content

chore: bump ethers#1598

Merged
mattsse merged 2 commits intomainfrom
dan/bump-ethers
Mar 1, 2023
Merged

chore: bump ethers#1598
mattsse merged 2 commits intomainfrom
dan/bump-ethers

Conversation

@Rjected
Copy link
Member

@Rjected Rjected commented Mar 1, 2023

Bump ethers to fix can_peer_geth in CI

@Rjected Rjected requested a review from gakonst as a code owner March 1, 2023 19:33
@codecov-commenter
Copy link

codecov-commenter commented Mar 1, 2023

Codecov Report

Merging #1598 (1e4ea30) into main (2884eae) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head 1e4ea30 differs from pull request most recent head fe71b67. Consider uploading reports for the commit fe71b67 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1598      +/-   ##
==========================================
- Coverage   74.81%   74.79%   -0.03%     
==========================================
  Files         374      374              
  Lines       44249    44249              
==========================================
- Hits        33105    33095      -10     
- Misses      11144    11154      +10     
Flag Coverage Δ
integration-tests 21.11% <ø> (-0.02%) ⬇️
unit-tests 69.43% <ø> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
crates/net/network/src/session/mod.rs 76.59% <0.00%> (-1.36%) ⬇️
crates/net/downloaders/src/headers/task.rs 83.67% <0.00%> (-1.03%) ⬇️
crates/stages/src/stages/sender_recovery.rs 90.95% <0.00%> (-0.54%) ⬇️
crates/net/eth-wire/src/ethstream.rs 83.97% <0.00%> (-0.48%) ⬇️
crates/stages/src/stages/bodies.rs 92.43% <0.00%> (-0.43%) ⬇️
crates/net/discv4/src/lib.rs 65.82% <0.00%> (+0.14%) ⬆️
crates/stages/src/stages/hashing_storage.rs 96.25% <0.00%> (+0.26%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mattsse mattsse added the A-dependencies Pull requests or issues that are about dependencies label Mar 1, 2023
@mattsse mattsse merged commit 6136e0d into main Mar 1, 2023
@mattsse mattsse deleted the dan/bump-ethers branch March 1, 2023 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-dependencies Pull requests or issues that are about dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants