★ wanayoo — archive 1999 https://github.com/nodejs/node/pull/18439Nouvelle recherche | Portail wanayoo
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: remove removed apis from http2 docs #18439

Closed
wants to merge 2 commits into from
Closed

Conversation

Copy link
Contributor

@kjin kjin commented Jan 29, 2018

  • Removed references to removed socketError event
  • Removed second definition of Http2Session#close (as far as I could tell in the implementation, close no longer accepts an options object)
  • Add parameter types for ClientHttp2Session 'altsvc' event
  • Re-ordered event names in Http2SecureServer to be alphabetical and added 'session' event (from Http2Server)
Checklist
Affected core subsystem(s)

doc

@nodejs-github-bot nodejs-github-bot added doc http2 labels Jan 29, 2018
@maclover7 maclover7 requested a review from apapirovski Jan 31, 2018
@BridgeAR BridgeAR added the author ready label Feb 1, 2018
@BridgeAR
Copy link
Member

@BridgeAR BridgeAR commented Feb 1, 2018

@jasnell
Copy link
Member

@jasnell jasnell commented Feb 2, 2018

Looks like some linting issues.

@kjin
Copy link
Contributor Author

@kjin kjin commented Feb 3, 2018

Oops, forgot to remove a ```. Thanks!

@addaleax
Copy link
Member

@addaleax addaleax commented Feb 4, 2018

Landed in 71b15e8

@addaleax addaleax closed this Feb 4, 2018
addaleax pushed a commit that referenced this issue Feb 4, 2018
PR-URL: #18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
@addaleax addaleax removed the author ready label Feb 4, 2018
MylesBorins pushed a commit that referenced this issue Feb 21, 2018
PR-URL: #18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
MylesBorins pushed a commit that referenced this issue Feb 21, 2018
PR-URL: #18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
MylesBorins pushed a commit that referenced this issue Feb 21, 2018
PR-URL: #18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
@MylesBorins MylesBorins mentioned this pull request Feb 21, 2018
@MylesBorins
Copy link
Member

@MylesBorins MylesBorins commented Mar 20, 2018

Should this be backported to v8.x-staging? If yes please follow the guide and raise a backport PR, if not let me know or add the dont-land-on label.

kjin added a commit to kjin/node that referenced this issue May 1, 2018
PR-URL: nodejs#18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
MylesBorins pushed a commit that referenced this issue May 2, 2018
Backport-PR-URL: #20456
PR-URL: #18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
@MylesBorins MylesBorins mentioned this pull request May 2, 2018
MayaLekova pushed a commit to MayaLekova/node that referenced this issue May 8, 2018
PR-URL: nodejs#18439
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc http2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants