FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[v16.x backport] src: return Maybe<bool> from InitializeContextRuntime() by RaisinTen · Pull Request #39834 · nodejs/node · GitHub

/ node Public

[v16.x backport] src: return Maybe<bool> from InitializeContextRuntime() - #39834

Closed
RaisinTen wants to merge 1 commit into
nodejs:v16.x-stagingfrom
RaisinTen:backport-39695-to-v16.x
Closed

[v16.x backport] src: return Maybe<bool> from InitializeContextRuntime()#39834
RaisinTen wants to merge 1 commit into
nodejs:v16.x-stagingfrom
RaisinTen:backport-39695-to-v16.x

Conversation

Copy link
Copy Markdown
Member

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

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>
nodejs-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. v16.x labels Aug 21, 2021

Copy link
Copy Markdown
Collaborator

RaisinTen requested review from addaleax and jasnell August 21, 2021 14:52
targos added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 22, 2021

targos commented Aug 22, 2021

Copy link
Copy Markdown
Member

Nits about the commit message (I'll fix while landing):

  • The [v16.x backport] should only be added to the PR title, not to the commit message
  • Backport-PR-URL is added while landing (with git node land --backport)

Basically, when you backport a PR, you only need to git cherry-pick the commits and fix the conflicts, keeping the original message.

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 22, 2021

This comment has been minimized.

RaisinTen added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Aug 22, 2021

This comment has been minimized.

Copy link
Copy Markdown
Collaborator

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>

targos commented Aug 23, 2021

Copy link
Copy Markdown
Member

Landed in 3a8399e

targos closed this Aug 23, 2021
RaisinTen deleted the backport-39695-to-v16.x branch August 23, 2021 14:56
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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 outstanding review comments, and a CI started. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL