Skip to content

chore: Simple refactor networks config #3361

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

Closed
wants to merge 4 commits into from

Conversation

Ivansete-status
Copy link
Collaborator

@Ivansete-status Ivansete-status commented Apr 8, 2025

Description

Simple PR to correct wrong type naming.
Thanks @fryorcraken for the suggestion!

note: aside from the original PR's purpose, I'm disabling a flaky test in test_waku_discv5, which was recently added. It will only be useful to play with it manually but I want to avoid more noise in our CI's

Issue

Copy link

github-actions bot commented Apr 8, 2025

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:3361

Built from 97addf6

@fryorcraken
Copy link
Collaborator

@Ivansete-status can we hold off until #3357 is done and merged please? I can add that to my todo list #3349

@Ivansete-status
Copy link
Collaborator Author

@Ivansete-status can we hold off until #3357 is done and merged please? I can add that to my todo list #3349

Sure , no problem. Thanks :)!

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