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

Fix pinning system on devel #820

Merged
merged 3 commits into from
Dec 14, 2022
Merged

Fix pinning system on devel #820

merged 3 commits into from
Dec 14, 2022

Conversation

Menduist
Copy link
Contributor

devel switched to new nimble, so the pinning system needs to be updated.

Now, the .pinned & nimble.lock need to be kept in sync, for more fun
Once we stop supporting nim versions with the old nimble, we'll be able to only use the nimble lock system

@codecov
Copy link

codecov bot commented Dec 13, 2022

Codecov Report

Merging #820 (a09af50) into unstable (a17cad7) will increase coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           unstable     #820      +/-   ##
============================================
+ Coverage     83.62%   83.63%   +0.01%     
============================================
  Files            81       81              
  Lines         14726    14726              
============================================
+ Hits          12314    12316       +2     
+ Misses         2412     2410       -2     
Impacted Files Coverage Δ
libp2p/protocols/secure/secure.nim 71.15% <0.00%> (-3.85%) ⬇️
libp2p/muxers/mplex/mplex.nim 88.23% <0.00%> (-0.85%) ⬇️
libp2p/crypto/rsa.nim 79.18% <0.00%> (-0.20%) ⬇️
libp2p/transports/wstransport.nim 85.26% <0.00%> (ø)
libp2p/protocols/pubsub/pubsubpeer.nim 87.87% <0.00%> (+2.27%) ⬆️
libp2p/muxers/mplex/lpchannel.nim 86.74% <0.00%> (+3.01%) ⬆️

@Menduist Menduist merged commit fe7a69e into unstable Dec 14, 2022
@Menduist Menduist deleted the fixpindevel branch December 14, 2022 11:10
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.

1 participant