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: use stronger language about security of vm #23198

Merged
merged 1 commit into from
Oct 4, 2018
Merged

doc: use stronger language about security of vm #23198

merged 1 commit into from
Oct 4, 2018

Conversation

devsnek
Copy link
Member

@devsnek devsnek commented Oct 1, 2018 •
edited
Loading

Try to use some more explicit language about the nature of VM

cc @nodejs/vm

Checklist

@nodejs-github-bot
Copy link
Collaborator

@devsnek build started: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/1087/pipeline

@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 Oct 1, 2018
@devsnek
Copy link
Member Author

devsnek commented Oct 4, 2018

ci lite https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/1120/

@devsnek
doc: use stronger language about security of vm
6d32d44 
PR-URL: nodejs#23198
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ujjwal Sharma <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
@devsnek
Copy link
Member Author

devsnek commented Oct 4, 2018

Landed in 6d32d44

@devsnek devsnek closed this Oct 4, 2018
@devsnek devsnek merged commit 6d32d44 into nodejs:master Oct 4, 2018
@devsnek devsnek deleted the hp2 branch October 4, 2018 06:31
targos pushed a commit that referenced this pull request Oct 5, 2018
@devsnek @targos
doc: use stronger language about security of vm
ed01b38 
PR-URL: #23198
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ujjwal Sharma <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
@targos targos mentioned this pull request Oct 7, 2018
Merged
This was referenced Oct 10, 2018
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
jasnell pushed a commit that referenced this pull request Oct 17, 2018
@devsnek @jasnell
doc: use stronger language about security of vm
e4a1683 
PR-URL: #23198
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ujjwal Sharma <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@trivikr trivikr trivikr approved these changes

@ryzokuken ryzokuken ryzokuken approved these changes

@BridgeAR BridgeAR BridgeAR approved these changes

@cjihrig cjihrig cjihrig approved these changes

@jasnell jasnell jasnell approved these changes

@bnoordhuis bnoordhuis bnoordhuis approved these changes

@jdalton jdalton jdalton approved these changes

Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations. vm Issues and PRs related to the vm subsystem.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@devsnek @nodejs-github-bot @jdalton @bnoordhuis @jasnell @cjihrig @BridgeAR @ryzokuken @trivikr