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: corrected version http.createServer options addition #25001

Closed
wants to merge 1 commit into from

Conversation

benswinburne
Copy link
Contributor

8.12 Changelog
https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V8.md#8.12.0

Specifically
[01dc646] - (SEMVER-MINOR) http: add options to http.createServer() (Peter Marton) #15752

Pull
#15752

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem. labels Dec 12, 2018
@benswinburne benswinburne changed the title Correct version whereby options was added to http.createServer doc: corrected version http.createServer options addition Dec 12, 2018
@mscdex
Copy link
Contributor

mscdex commented Dec 13, 2018

Both versions should be documented as this feature was backported from v9.x to v8.x.

@benswinburne
Copy link
Contributor Author

Surely the documented version is since as opposed to a list of versions?

If I am to list multiple, do you suggest it lists both in the same way.

changes:
  - version: v9.6.0
    pr-url: https://github.com/nodejs/node/pull/15752
    description: The `options` argument is supported now.
  - version: v8.12.0
    pr-url: https://github.com/nodejs/node/pull/15752
    description: The `options` argument is supported now.

or explicitly note it was backported

changes:
  - version: v9.6.0
    pr-url: https://github.com/nodejs/node/pull/15752
    description: The `options` argument is supported now.
  - version: v8.12.0
    pr-url: https://github.com/nodejs/node/pull/15752
    description: The `options` argument was backported.

Perhaps you could suggest an example of another backported feature I can reference?

@mscdex
Copy link
Contributor

mscdex commented Dec 13, 2018

You can use comma-separated lists, like (newest versions first):

- version: v9.6.0, v8.12.0

That's how we're currently handling this in other parts of the documentation.

@lpinca
Copy link
Member

lpinca commented Mar 11, 2019

@nodejs/documentation

@lpinca
Copy link
Member

lpinca commented Mar 11, 2019

@vsemozhetbyt vsemozhetbyt added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 11, 2019
@vsemozhetbyt
Copy link
Contributor

Landed in 9a39a65
Thank you. Sorry for the delay.

BridgeAR pushed a commit that referenced this pull request Mar 13, 2019
8.12 Changelog
https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V8.md#8.12.0

Specifically
[01dc646] - http: add options to http.createServer() #15752

PR-URL: #25001
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
BridgeAR pushed a commit that referenced this pull request Mar 14, 2019
8.12 Changelog
https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V8.md#8.12.0

Specifically
[01dc646] - http: add options to http.createServer() #15752

PR-URL: #25001
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
BethGriggs pushed a commit that referenced this pull request Apr 16, 2019
8.12 Changelog
https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V8.md#8.12.0

Specifically
[01dc646] - http: add options to http.createServer() #15752

PR-URL: #25001
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
@BethGriggs BethGriggs mentioned this pull request May 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. 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