doc: Fix bug in SimpleProtocol example code #5640

wants to merge 1 commit into


None yet

4 participants

Jimbly commented Jun 7, 2013

No description provided.


Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

Commit Jimbly/node@c768728 has the following error(s):

  • Commit message must indicate the subsystem this commit changes

You can fix all these things without opening another issue.

Please see for more information

a0viedo commented Dec 10, 2014

Hey @Jimbly could you re-submit this PR to Someone could adopt it for you if you are not interested.

@jasnell jasnell closed this Aug 6, 2015
Jimbly commented Aug 6, 2015

If by "already", you mean "6 months after this PR was submitted"... good to see it's fixed though!

jasnell commented Aug 6, 2015

;-) ... "If I say I'm going to do it, there's no need to nag me about it every six months. It'll get done eventually"

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