Skip to content

src: return Maybe<bool> from InitializeContextRuntime() - #39695

Closed
RaisinTen wants to merge 1 commit into
nodejs:masterfrom
RaisinTen:src/return-Maybe-bool-fromInitializeContextRuntime
Closed

src: return Maybe<bool> from InitializeContextRuntime()#39695
RaisinTen wants to merge 1 commit into
nodejs:masterfrom
RaisinTen:src/return-Maybe-bool-fromInitializeContextRuntime

Conversation

@RaisinTen

Copy link
Copy Markdown
Member

Signed-off-by: Darshan Sen darshan.sen@postman.com

@RaisinTen
RaisinTen requested a review from addaleaxAugust 7, 2021 08:35
@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Aug 7, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@addaleaxaddaleax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

Comment threadsrc/api/environment.cc Outdated
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
@RaisinTen
RaisinTenforce-pushed the src/return-Maybe-bool-fromInitializeContextRuntime branch from a42dadf to 5bd551aCompareAugust 7, 2021 10:03
@addaleaxaddaleax added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed needs-ci PRs that need a full CI run. labels Aug 7, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

jasnell pushed a commit that referenced this pull request Aug 12, 2021
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
PR-URL: #39695
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell

Copy link
Copy Markdown
Member

Landed in 284f164

@jasnelljasnell closed this Aug 12, 2021
@RaisinTen
RaisinTen deleted the src/return-Maybe-bool-fromInitializeContextRuntime branch August 13, 2021 11:00
@danielleadams

Copy link
Copy Markdown
Contributor

@RaisinTen do you mind backporting this to v16.x-staging? It did not land cleanly. Thanks!

RaisinTen added a commit to RaisinTen/node that referenced this pull request Aug 21, 2021
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
PR-URL: nodejs#39695
Backport-PR-URL: nodejs#39834
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Aug 23, 2021
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
PR-URL: #39695
Backport-PR-URL: #39834
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targostargos mentioned this pull request Aug 25, 2021
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.c++Issues and PRs that require attention from people who are familiar with C++.lib / srcIssues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@RaisinTen@nodejs-github-bot@jasnell@danielleadams@addaleax@targos