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: http.IncomingMessage.statusMessage description out of place #4558

Closed
jmm opened this issue Jan 6, 2016 · 2 comments
Closed

doc: http.IncomingMessage.statusMessage description out of place #4558

jmm opened this issue Jan 6, 2016 · 2 comments
Labels
doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem.

Comments

@jmm
Copy link
Contributor

jmm commented Jan 6, 2016

statusMessage description lost its place and is under socket.

See f4c259a / #3662.
@mscdex mscdex added http Issues or PRs related to the http subsystem. doc Issues and PRs related to the documentations. labels Jan 7, 2016
@ryansobol
Copy link
Contributor

Care to make a PR? :)

techjeffharris added a commit to techjeffharris/node that referenced this issue Jan 22, 2016
@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
Copy link
Contributor

I am running tests for a PR at the moment.

@techjeffharris techjeffharris mentioned this issue Jan 22, 2016
Closed
rvagg pushed a commit that referenced this issue 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 <[email protected]>
Reviewed-By: Roman Klauke <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
benjamingr pushed a commit to benjamingr/io.js that referenced this issue 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 <[email protected]>
Reviewed-By: Roman Klauke <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this issue 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 <[email protected]>
Reviewed-By: Roman Klauke <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this issue 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 <[email protected]>
Reviewed-By: Roman Klauke <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this issue 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 <[email protected]>
Reviewed-By: Roman Klauke <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
scovetta pushed a commit to scovetta/node that referenced this issue 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 <[email protected]>
Reviewed-By: Roman Klauke <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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

No branches or pull requests
4 participants
@ryansobol @mscdex @jmm @techjeffharris