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.

Sign up for GitHub

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

Jump to bottom

doc: undo move http.IncomingMessage.statusMessage #4822

Closed
wants to merge 1 commit into from
Closed

doc: undo move http.IncomingMessage.statusMessage #4822

wants to merge 1 commit into from

Conversation

techjeffharris
Copy link
Contributor

The description for statusMessage was accidentally moved under the
heading for http.IncomingMessage.socket. This commit puts it back
in the correct place.

/CC @nodejs/http @nodejs/documentation

Fixes: #4558

@techjeffharris
doc: undo move http.IncomingMessage.statusMessage
2af0901 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

see: nodejs#4558
@techjeffharris techjeffharris mentioned this pull request Jan 22, 2016
Closed
@jasnell
Copy link
Member

jasnell commented Jan 23, 2016

LGTM

@jasnell jasnell added doc Issues and PRs related to the documentations. lts-watch-v4.x labels Jan 23, 2016
@mscdex mscdex added the http Issues or PRs related to the http subsystem. label Jan 23, 2016
@r-52
Copy link
Contributor

r-52 commented Jan 23, 2016

LGTM

@silverwind
Copy link
Contributor

Thanks! Landed in f19b9d6.

silverwind pushed a commit that referenced this pull request Jan 24, 2016
@techjeffharris @silverwind
doc: undo move http.IncomingMessage.statusMessage
f19b9d6 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: #4822
Fixes: #4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
@silverwind silverwind closed this Jan 24, 2016
rvagg pushed a commit that referenced this pull request Jan 25, 2016
@techjeffharris @rvagg
doc: undo move http.IncomingMessage.statusMessage
f462320 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: #4822
Fixes: #4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
@abdulhannanali
Copy link

LGTM!

benjamingr pushed a commit to benjamingr/io.js that referenced this pull request Jan 27, 2016
@techjeffharris @benjamingr
doc: undo move http.IncomingMessage.statusMessage
5f8b49d 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: nodejs#4822
Fixes: nodejs#4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
@rvagg rvagg mentioned this pull request Feb 8, 2016
Closed
MylesBorins pushed a commit that referenced this pull request Feb 17, 2016
@techjeffharris
doc: undo move http.IncomingMessage.statusMessage
d8fbc28 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: #4822
Fixes: #4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
MylesBorins pushed a commit that referenced this pull request Feb 18, 2016
@techjeffharris
doc: undo move http.IncomingMessage.statusMessage
b2d2c0b 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: #4822
Fixes: #4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
@MylesBorins MylesBorins mentioned this pull request Feb 18, 2016
Merged
MylesBorins pushed a commit that referenced this pull request Mar 2, 2016
@techjeffharris
doc: undo move http.IncomingMessage.statusMessage
242d3bc 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: #4822
Fixes: #4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
@techjeffharris
doc: undo move http.IncomingMessage.statusMessage
6a13c1c 
The description for `statusMessage` was accidentally moved under the
heading for `http.IncomingMessage.socket`.  This commit puts it back
in the correct place.

PR-URL: nodejs#4822
Fixes: nodejs#4558
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
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.
7 participants
@techjeffharris @jasnell @r-52 @silverwind @abdulhannanali @mscdex @MylesBorins