Skip to content
New issue

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

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

Already on GitHub? # to your account

chore: stop using the deprecated github.com/libp2p/go-libp2p-yamux #8998

Merged
merged 1 commit into from
Jul 8, 2022

Conversation

marten-seemann
Copy link
Contributor

Followup to #8996.

Looks like the reason for these fixes is that the update procedure as described in Release Notes was not followed (and apparently Lotus is not running staticcheck on CI).

@marten-seemann marten-seemann requested a review from a team as a code owner July 8, 2022 09:24
@codecov
Copy link

codecov bot commented Jul 8, 2022

Codecov Report

Merging #8998 (1763d15) into master (852b5c5) will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8998      +/-   ##
==========================================
- Coverage   40.68%   40.62%   -0.06%     
==========================================
  Files         707      707              
  Lines       78716    78716              
==========================================
- Hits        32022    31976      -46     
- Misses      41241    41267      +26     
- Partials     5453     5473      +20     
Impacted Files Coverage Δ
node/modules/lp2p/smux.go 9.09% <ø> (ø)
chain/consensus/filcns/weight.go 70.58% <0.00%> (-5.89%) ⬇️
chain/stmgr/execute.go 91.30% <0.00%> (-4.35%) ⬇️
chain/stmgr/supply.go 63.77% <0.00%> (-3.94%) ⬇️
storage/wdpost/wdpost_sched.go 81.90% <0.00%> (-3.81%) ⬇️
storage/pipeline/currentdealinfo.go 73.80% <0.00%> (-2.39%) ⬇️
chain/consensus/filcns/mine.go 66.66% <0.00%> (-2.39%) ⬇️
chain/actors/builtin/power/power.go 82.60% <0.00%> (-2.18%) ⬇️
markets/storageadapter/client.go 40.55% <0.00%> (-1.85%) ⬇️
chain/store/store.go 62.83% <0.00%> (-1.84%) ⬇️
... and 24 more

@Stebalien Stebalien merged commit a825682 into master Jul 8, 2022
@Stebalien Stebalien deleted the fix-yamux-option branch July 8, 2022 16:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants