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

fix(sepcs/p2p): link to libp2p Yamux specification #3493

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

mxinden
Copy link
Contributor

@mxinden mxinden commented Sep 4, 2023

Previously the specification would link to the Hashicorp Yamux specification.

https://github.com/hashicorp/yamux/blob/master/spec.md

Since adoption by libp2p, there have been multiple refinements to the original specification. The improved specification can be found in the libp2p specification repository.

https://github.com/libp2p/specs/blob/master/yamux/README.md

//CC @AgeManning and @jxs

Copy link
Member

@ppopth ppopth left a comment

Choose a reason for hiding this comment

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

LGTM!

@dapplion
Copy link
Collaborator

@mxinden can you rebase your PR on latest dev? CI is failing and needs the fix from #3495

Previously the specification would link to the Hashicorp Yamux specification.

https://github.com/hashicorp/yamux/blob/master/spec.md

Since adoption by libp2p, there have been multiple refinements to the original specification. The improved specification can be found in the libp2p specification repository.

https://github.com/libp2p/specs/blob/master/yamux/README.md
@mxinden
Copy link
Contributor Author

mxinden commented Sep 14, 2023

@dapplion done. Mind taking another look?

@hwwhww hwwhww added scope:networking general:presentation Presentation (as opposed to content) labels Sep 15, 2023
@hwwhww hwwhww mentioned this pull request Sep 15, 2023
5 tasks
Copy link
Contributor

@hwwhww hwwhww left a comment

Choose a reason for hiding this comment

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

Thank you @mxinden!

@hwwhww hwwhww merged commit d26dac9 into ethereum:dev Sep 15, 2023
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
general:presentation Presentation (as opposed to content) scope:networking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants