Skip to content

Conversation

@RaisinTen
Copy link
Member

@RaisinTenRaisinTen commented Nov 15, 2020

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Nov 15, 2020
@RaisinTenRaisinTen marked this pull request as ready for review November 15, 2020 16:41
@addaleaxaddaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 15, 2020
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 15, 2020
@nodejs-github-bot
Copy link
Collaborator

@danbev
Copy link
Contributor

Re-run of failing node-test-commit-linux-containered ✔️

@danbev
Copy link
Contributor

Landed in 091f0d1.

@danbevdanbev closed this Nov 19, 2020
danbev pushed a commit that referenced this pull request Nov 19, 2020
PR-URL: #36128 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
@RaisinTenRaisinTen deleted the refactor/node_env_var.cc branch November 19, 2020 08:43
codebytere pushed a commit that referenced this pull request Nov 22, 2020
PR-URL: #36128 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
@codebyterecodebytere mentioned this pull request Nov 22, 2020
BethGriggs pushed a commit that referenced this pull request Dec 10, 2020
PR-URL: #36128 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
BethGriggs pushed a commit that referenced this pull request Dec 10, 2020
PR-URL: #36128 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
@BethGriggsBethGriggs mentioned this pull request Dec 10, 2020
BethGriggs pushed a commit that referenced this pull request Dec 15, 2020
PR-URL: #36128 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RaisinTen@nodejs-github-bot@danbev@Trott@addaleax