Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: update go-libp2p@v0.19 #8511

Merged
merged 2 commits into from
Apr 22, 2022
Merged

deps: update go-libp2p@v0.19 #8511

merged 2 commits into from
Apr 22, 2022

Conversation

vyzo
Copy link
Contributor

@vyzo vyzo commented Apr 19, 2022

libp2p v0.19 has been released

@vyzo vyzo requested a review from magik6k April 19, 2022 16:30
@vyzo vyzo requested a review from a team as a code owner April 19, 2022 16:30
@jennijuju
Copy link
Member

Ci is sad + please follow the pr title format mentioned in pr template !

@vyzo
Copy link
Contributor Author

vyzo commented Apr 20, 2022

CI is sad because of go version; waiting for #8512

@vyzo vyzo changed the title update go-libp2p@v0.19 deps: update go-libp2p@v0.19 Apr 20, 2022
@vyzo vyzo force-pushed the deps/update-libp2p branch from 8fedfb2 to b00f8cb Compare April 20, 2022 14:17
@vyzo
Copy link
Contributor Author

vyzo commented Apr 20, 2022

the lint failure is an artifact of the new linter version; i think we should ignore it in this pr and fix it in a separate one.

@jennijuju
Copy link
Member

@vyzo lets fix it in this PR as it introduces it? (the later PRs aren't facing this failure.

@vyzo
Copy link
Contributor Author

vyzo commented Apr 21, 2022

yes, just needs rebase

@jennijuju jennijuju force-pushed the deps/update-libp2p branch from 5c8d888 to cc29449 Compare April 22, 2022 00:20
@jennijuju
Copy link
Member

yes, just needs rebase

rebased happy lint

@codecov
Copy link

codecov bot commented Apr 22, 2022

Codecov Report

Merging #8511 (cc29449) into master (bab500b) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8511   +/-   ##
=======================================
  Coverage   40.79%   40.79%           
=======================================
  Files         686      686           
  Lines       75582    75582           
=======================================
+ Hits        30831    30834    +3     
+ Misses      39428    39415   -13     
- Partials     5323     5333   +10     
Impacted Files Coverage Δ
chain/exchange/peer_tracker.go 66.66% <0.00%> (-4.31%) ⬇️
chain/events/events_called.go 83.90% <0.00%> (-3.42%) ⬇️
chain/stmgr/call.go 69.18% <0.00%> (-3.25%) ⬇️
chain/sync_manager.go 73.91% <0.00%> (-2.80%) ⬇️
chain/stmgr/searchwait.go 67.30% <0.00%> (-2.57%) ⬇️
storage/wdpost_sched.go 79.41% <0.00%> (-1.97%) ⬇️
extern/storage-sealing/currentdealinfo.go 70.17% <0.00%> (-1.76%) ⬇️
blockstore/autobatch.go 56.30% <0.00%> (-1.69%) ⬇️
extern/sector-storage/sched.go 84.41% <0.00%> (-1.53%) ⬇️
miner/miner.go 56.72% <0.00%> (-1.32%) ⬇️
... and 14 more

@jennijuju jennijuju merged commit ff987e0 into master Apr 22, 2022
@jennijuju jennijuju deleted the deps/update-libp2p branch April 22, 2022 00:44
@jennijuju jennijuju mentioned this pull request Apr 22, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants