FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: add HTTP working group by jasnell · Pull Request #3919 · nodejs/node · GitHub

/ node Public

doc: add HTTP working group - #3919

Closed
jasnell wants to merge 2 commits into
nodejs:masterfrom
jasnell:http-wg-charter
ConversationCommits (2)ChecksFiles changed
Closed

doc: add HTTP working group#3919
jasnell wants to merge 2 commits into
nodejs:masterfrom
jasnell:http-wg-charter

Conversation

jasnell commented Nov 19, 2015

Copy link
Copy Markdown
Member

Proposed charter for the HTTP Working Group

@nodejs/http @nodejs/ctc

jasnell mentioned this pull request Nov 19, 2015
Open
cjihrig reviewed Nov 19, 2015
View reviewed changes
Comment thread WORKING_GROUPS.md Outdated

cjihrig Nov 19, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

cjihrig commented Nov 19, 2015

Copy link
Copy Markdown
Contributor

LGTM with one small comment.

doc: add HTTP working group
14cab8f 
Proposed charter for the HTTP Working Group

jasnell commented Nov 19, 2015

Copy link
Copy Markdown
Member Author

@cjihrig .. fixed and updated.

mscdex commented Nov 19, 2015

Copy link
Copy Markdown
Contributor

LGTM

mscdex added the doc Issues and PRs related to the documentations. label Nov 19, 2015
cjihrig reviewed Nov 19, 2015
View reviewed changes
Comment thread WORKING_GROUPS.md Outdated

cjihrig Nov 19, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

jasnell Nov 19, 2015

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

odd, ok will fix!

mscdex added the http Issues or PRs related to the http subsystem. label Nov 19, 2015

jasnell commented Nov 19, 2015

Copy link
Copy Markdown
Member Author

@cjihrig .. fixed

dougwilson commented Nov 20, 2015

Copy link
Copy Markdown
Member

LGTM

jasnell commented Nov 20, 2015

Copy link
Copy Markdown
Member Author

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
doc: add HTTP working group
2b1ecfe 
Charter for the HTTP Working Group

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

jasnell commented Dec 3, 2015

Copy link
Copy Markdown
Member Author

Landed in 2b1ecfe

jasnell closed this Dec 3, 2015
jasnell added a commit that referenced this pull request Dec 5, 2015
doc: add HTTP working group
85b6012 
Charter for the HTTP Working Group

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

PR-URL: #3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
jasnell mentioned this pull request Dec 17, 2015
Closed
jasnell added a commit that referenced this pull request Dec 17, 2015
doc: add HTTP working group
f93d268 
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
doc: add HTTP working group
de39163 
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
doc: add HTTP working group
e4fc0c7 
Charter for the HTTP Working Group

PR-URL: nodejs#3919
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
williamkapke mentioned this pull request Apr 18, 2016
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

Back | FazBrowse Home | New Git URL