-
Notifications
You must be signed in to change notification settings - Fork 41
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
Gossipsub v1.1 #82
Gossipsub v1.1 #82
Conversation
Codecov Report
@@ Coverage Diff @@
## master #82 +/- ##
===========================================
+ Coverage 80.00% 91.66% +11.66%
===========================================
Files 1 2 +1
Lines 10 24 +14
===========================================
+ Hits 8 22 +14
Misses 2 2
Continue to review full report at Codecov.
|
@wemeetagain me and @jacobheunhave been adopting smaller PRs rather than big changes lately, in order to ease reviews. Perhaps we can do something similar here, what do you think? Divisions that make sense of course, like dividing by parts that work for themselves and we merge everything to a branch. When everything is ready, we can merge to master. Is this reasonable for you? |
Thats a great idea. Definitely a fan of the smaller PRs. |
gs1.1 constants and comments
Co-authored-by: Vasco Santos <vasco.santos@moxy.studio>
gs1.1 peer score
chore: use pubsub interface
gs1.1: chore: add go gossipsub tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Open draft PR for visibility