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

doc: add HTTP working group #3919

Closed
wants to merge 2 commits into from
Closed

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Nov 19, 2015

Proposed charter for the HTTP Working Group

@nodejs/http @nodejs/ctc

* Authoring and editing HTTP documentation within the Node.js project.
* Reviewing changes to HTTP functionality within the Node.js project.
* Working with the ecosystem of HTTP related module developers to evolve the
HTTP implementation and APIs in core
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you use consistent periods or consistent no periods at the end of these points.

@cjihrig
Copy link
Contributor

cjihrig commented Nov 19, 2015

LGTM with one small comment.

Proposed charter for the HTTP Working Group
@jasnell
Copy link
Member Author

jasnell commented Nov 19, 2015

@cjihrig .. fixed and updated.

@mscdex
Copy link
Contributor

mscdex commented Nov 19, 2015

LGTM

@mscdex mscdex added the doc Issues and PRs related to the documentations. label Nov 19, 2015
@@ -29,9 +29,13 @@ back in to the TSC.
* [Addon API](#addon-api)
* [Benchmarking](#benchmarking)
* [Post-mortem](#post-mortem)
* [Intl](#Intl)
* [HTTP](#HTTP)
Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, I just tried this link in my browser and it didn't work. Lowercase http worked fine though. Same for Intl.

Copy link
Member Author

Choose a reason for hiding this comment

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

odd, ok will fix!

@mscdex mscdex added the http Issues or PRs related to the http subsystem. label Nov 19, 2015
@jasnell
Copy link
Member Author

jasnell commented Nov 19, 2015

@cjihrig .. fixed

@dougwilson
Copy link
Member

LGTM

@jasnell
Copy link
Member Author

jasnell commented Nov 20, 2015

ping @nodejs/ctc ...would like to get more review on this before landing

jasnell added a commit that referenced this pull request Dec 3, 2015
Charter for the HTTP Working Group

PR-URL: #3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
@jasnell
Copy link
Member Author

jasnell commented Dec 3, 2015

Landed in 2b1ecfe

@jasnell jasnell closed this Dec 3, 2015
jasnell added a commit that referenced this pull request Dec 5, 2015
Charter for the HTTP Working Group

PR-URL: #3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
jasnell added a commit that referenced this pull request Dec 15, 2015
Charter for the HTTP Working Group

PR-URL: #3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
@jasnell jasnell mentioned this pull request Dec 17, 2015
jasnell added a commit that referenced this pull request Dec 17, 2015
Charter for the HTTP Working Group

PR-URL: #3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
jasnell added a commit that referenced this pull request Dec 23, 2015
Charter for the HTTP Working Group

PR-URL: #3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
Charter for the HTTP Working Group

PR-URL: nodejs#3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants