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

chore: bumping dependencies for 0.27.0 #2572

Merged
merged 25 commits into from
Apr 9, 2024

Conversation

gabrielmer
Copy link
Contributor

@gabrielmer gabrielmer commented Apr 7, 2024

Description

Updating dependencies for release 0.27.0

Bumped Items

  • nim-bearssl
  • nimbus-build-system
  • nim-chronos
  • nim-faststreams
  • nim-json-serialization
  • nim-libbacktrace
  • nim-libp2p ( update to the unstable branch )
  • nim-presto
  • nim-stew
  • nim-stint
  • nim-unicodedb
  • nim-unittest2

Additional changes

For the bumping of nim-libp2p, we're now initializing the GossipSub parameters with its init proc so when a parameter is not set by us, the nim-libp2p's set default value is used. Before this change, if we didn't set a value, the corresponding type's default is used (0 for all the ints, for example).

Notes

Issue

closes #2570

@gabrielmer gabrielmer marked this pull request as draft April 7, 2024 11:55
Copy link

github-actions bot commented Apr 7, 2024

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:2572-rln-v2-true

Built from 0a4ebea

@gabrielmer gabrielmer self-assigned this Apr 7, 2024
@gabrielmer gabrielmer marked this pull request as ready for review April 8, 2024 18:39
Copy link
Collaborator

@Ivansete-status Ivansete-status left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for it! 💯

If not done already, maybe it worth to ping some of the chronicles maintainers to take a look at the issue:

<< nim-chronicles is not updated, because of same issue as mentioned in #2472 (comment) >>

Copy link
Contributor

@NagyZoltanPeter NagyZoltanPeter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's great. Thank you!

@gabrielmer
Copy link
Contributor Author

LGTM! Thanks for it! 💯

If not done already, maybe it worth to ping some of the chronicles maintainers to take a look at the issue:

<< nim-chronicles is not updated, because of same issue as mentioned in #2472 (comment) >>

100%! I'm contacting them :)

@gabrielmer gabrielmer merged commit f68ac79 into master Apr 9, 2024
13 of 15 checks passed
@gabrielmer gabrielmer deleted the chore-bumping-dependencies-for-0.27.0 branch April 9, 2024 08:17
gabrielmer added a commit that referenced this pull request Apr 25, 2024
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.

Bump vendor dependencies for release 0.27.0
3 participants