Skip to content

Update protocol spec to align with implementation#113

Open
CVTJNII wants to merge 1 commit into
logstash-plugins:mainfrom
CVTJNII:UpdateProtoDocumentation
Open

Update protocol spec to align with implementation#113
CVTJNII wants to merge 1 commit into
logstash-plugins:mainfrom
CVTJNII:UpdateProtoDocumentation

Conversation

@CVTJNII

@CVTJNII CVTJNII commented Jul 28, 2016

Copy link
Copy Markdown

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
Copy Markdown

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

@CVTJNII

CVTJNII commented Jul 28, 2016

Copy link
Copy Markdown
Author

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

@ph

ph commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

@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