Skip to content

doc: fix vm.runInNewContext signature - #21824

Closed
targos wants to merge 1 commit into
nodejs:masterfrom
targos:fix-vm-doc
Closed

doc: fix vm.runInNewContext signature#21824
targos wants to merge 1 commit into
nodejs:masterfrom
targos:fix-vm-doc

Conversation

@targos

Copy link
Copy Markdown
Member

The options parameter cannot be passed without sandbox.

Checklist

The `options` parameter cannot be passed without `sandbox`.
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. vm Issues and PRs related to the vm subsystem. labels Jul 15, 2018
@targostargos added the fast-track PRs that do not need to wait for 48 hours to land. label Jul 15, 2018
@targos

Copy link
Copy Markdown
MemberAuthor

Please 👍 this comment to fast-track.

@vsemozhetbytvsemozhetbyt added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 15, 2018
@addaleax

Copy link
Copy Markdown
Member

Landed in d279a8f

addaleax pushed a commit that referenced this pull request Jul 15, 2018
The `options` parameter cannot be passed without `sandbox`.
PR-URL: #21824
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
targos added a commit that referenced this pull request Jul 16, 2018
The `options` parameter cannot be passed without `sandbox`.
PR-URL: #21824
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@targostargos mentioned this pull request Jul 17, 2018
@targos
targos deleted the fix-vm-doc branch July 18, 2018 06:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.fast-trackPRs that do not need to wait for 48 hours to land.vmIssues and PRs related to the vm subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@targos@nodejs-github-bot@addaleax@lpinca@TimothyGu@vsemozhetbyt