Skip to content

test: enable gossipsub tests#3424

Open
dozyio wants to merge 13 commits intolibp2p:mainfrom
dozyio:test/enable-gossipsub-tests
Open

test: enable gossipsub tests#3424
dozyio wants to merge 13 commits intolibp2p:mainfrom
dozyio:test/enable-gossipsub-tests

Conversation

@dozyio
Copy link
Copy Markdown
Collaborator

@dozyio dozyio commented Mar 29, 2026

Description

Fixes and enables gossipsub tests

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if necessary (this includes comments as well)
  • I have added tests that prove my fix is effective or that my feature works

@dozyio dozyio changed the title test: enable gossipsub tests tests: enable gossipsub tests Mar 29, 2026
@dozyio dozyio changed the title tests: enable gossipsub tests test: enable gossipsub tests Mar 29, 2026
@dozyio dozyio marked this pull request as ready for review March 29, 2026 23:14
@dozyio dozyio requested a review from a team as a code owner March 29, 2026 23:14
@dozyio dozyio requested a review from tabcat March 29, 2026 23:19
@tabcat tabcat self-assigned this Apr 1, 2026
Copy link
Copy Markdown
Collaborator

@tabcat tabcat left a comment

Choose a reason for hiding this comment

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

🚀 🚀 🚀

can you summarize changes needed to support v3 interface? would me help understand changes more easily.

there are some places where code could be reused by making a function but up to you.

@dozyio
Copy link
Copy Markdown
Collaborator Author

dozyio commented Apr 4, 2026

🚀 🚀 🚀

can you summarize changes needed to support v3 interface? would me help understand changes more easily.

there are some places where code could be reused by making a function but up to you.

This PR covers the v3 changes https://github.com/ChainSafe/js-libp2p-gossipsub/pull/532/changes although plan is to bring gossipsub in-tree

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