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

Update protocol spec to align with implementation #113

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

CVTJNII
Copy link

@CVTJNII CVTJNII commented Jul 28, 2016

The following pull updates the protocol spec with the following changes:

Not sure if the CLA is required for simple documentation fixes, so please let me know if it is. Thanks.

@untergeek
Copy link

CLA is necessary for even documentation changes, yes. Please sign the CLA.

@CVTJNII
Copy link
Author

CVTJNII commented Jul 28, 2016

Okay, I'll work on getting it signed. I need corporate approval so it will probably take a little bit. Thanks.

@ph
Copy link
Contributor

ph commented Jul 29, 2016

@CVTJNII Changes LGTM, so I will gladly merge it when the CLA is signed.

@jsvd jsvd closed this Jan 23, 2020
@jsvd jsvd reopened this Jan 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants