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: bump vendors for 0.24.0 #2333

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Conversation

Ivansete-status
Copy link
Collaborator

Description

The following vendors have changes but are not being updated for the reason explained.

nim-web3: not updated because unit tests started to fail and no straightforward solution found.

nim-toml-serialization: not updated because it introduced a breaking change on how the --config-file attribute is parsed. The array attributes now need a comma. For example, the following attribute from within the config file:

pubsub-topic = [ "/waku/2/default-waku/proto" "/waku/2/testing-store" ]

... should be converted to:

pubsub-topic = [ "/waku/2/default-waku/proto", "/waku/2/testing-store" ]

and we cannot accept that breaking change

Issue

closes #2329

The following vendors have changes but are not being updated for
the reason explained.

nim-web3: not updated because unit tests started to fail and no
straightforward solution found.

nim-toml-serialization: not updated because it introduced a breaking
change on how the --config-file attribute is parsed. The array
attributes now need a comma. For example, the following attribute
from within the config file:

pubsub-topic = [ "/waku/2/default-waku/proto" "/waku/2/testing-store" ]

... should be converted to:

pubsub-topic = [ "/waku/2/default-waku/proto", "/waku/2/testing-store" ]

and we cannot accept that breaking change
Copy link

github-actions bot commented Jan 4, 2024

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:2333

Built from 7c7b8e5

@Ivansete-status Ivansete-status marked this pull request as ready for review January 4, 2024 14:47
Copy link
Contributor

@gabrielmer gabrielmer 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 so much! 🔥

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.

LGTM! Thanks for taking care!

@Ivansete-status Ivansete-status merged commit bd7570f into master Jan 4, 2024
10 checks passed
@Ivansete-status Ivansete-status deleted the bump-vendor-for-0.24.0 branch January 4, 2024 16:35
@Ivansete-status Ivansete-status self-assigned this Jan 8, 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.24.0
4 participants