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: add changes entry for vm code generation options #19440

Closed
wants to merge 2 commits into from

Conversation

TomCoded
Copy link
Contributor

Add changes entries for vm.createContext codeGeneration
option and script.runInNewContext contextCodeGeneration
option.

fixes: #19419
refs: #19016

Checklist

Add changes entries for vm.createContext codeGeneration
option and script.runInNewContext contextCodeGeneration
option.

fixes: nodejs#19419
refs: nodejs#19016
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. vm Issues and PRs related to the vm subsystem. labels Mar 19, 2018
@gibfahn gibfahn mentioned this pull request Mar 19, 2018
4 tasks
doc/api/vm.md Outdated
changes:
- version: REPLACEME
pr-url: https://github.com/nodejs/node/pull/19016
description: The `contextCodeGeneration` option is supported now.
Copy link
Member

Choose a reason for hiding this comment

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

Nit: can you please keep this aligned with pr-url? Shift left by 4 spaces.

doc/api/vm.md Outdated
changes:
- version: REPLACEME
pr-url: https://github.com/nodejs/node/pull/19016
description: The `codeGeneration` option is supported now.
Copy link
Member

Choose a reason for hiding this comment

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

Ditto.

Copy link
Contributor

@cjihrig cjihrig left a comment

Choose a reason for hiding this comment

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

LGTM with nits addressed.

@lpinca
Copy link
Member

lpinca commented Mar 20, 2018

@lpinca lpinca added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 20, 2018
@tniessen
Copy link
Member

Landed in 54fbbb1, thank you!

@tniessen tniessen closed this Mar 25, 2018
tniessen pushed a commit that referenced this pull request Mar 25, 2018
Add changes entries for vm.createContext codeGeneration
option and script.runInNewContext contextCodeGeneration
option.

PR-URL: #19440
Fixes: #19419
Refs: #19016
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos targos added dont-land-on-v4.x and removed author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Mar 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. vm Issues and PRs related to the vm subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doc: missing "changes" entry for vm code generation options
9 participants