Skip to content
This repository was archived by the owner on Jul 22, 2026. It is now read-only.

fix: in order for the isPubSub function to work with gossipsub this is mandatory - #512

Open
sfroment wants to merge 2 commits into
ChainSafe:masterfrom
sfroment:master
Open

fix: in order for the isPubSub function to work with gossipsub this is mandatory#512
sfroment wants to merge 2 commits into
ChainSafe:masterfrom
sfroment:master

Conversation

@sfroment

Copy link
Copy Markdown

Signed-off-by: Sacha Froment sfroment42@gmail.com

…s mandatory

Signed-off-by: Sacha Froment <sfroment42@gmail.com>
@sfroment
sfroment requested a review from a team as a code owner January 11, 2025 10:43
achingbrain
achingbrain previously approved these changes Jan 13, 2025
@achingbrain

Copy link
Copy Markdown
Collaborator

Small nit but it might be worth adding a test that asserts instances of GosspSub make isPubSub return true?

@sfroment

sfroment commented Jan 13, 2025

Copy link
Copy Markdown
Author

Will add and won't forget such stuff in the future 🙏
EDIT: added 👌

Signed-off-by: Sacha Froment <sfroment42@gmail.com>
@sfroment

Copy link
Copy Markdown
Author

@wemeetagain

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants