Skip to content

Conversation

RaisinTen
Copy link
Member

@RaisinTen RaisinTen commented Nov 15, 2020

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

@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Nov 15, 2020
@RaisinTen RaisinTen marked this pull request as ready for review November 15, 2020 16:41
@addaleax addaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 15, 2020
@github-actions github-actions bot 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

danbev commented Nov 19, 2020

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

@danbev
Copy link
Contributor

danbev commented Nov 19, 2020

Landed in 091f0d1.

@danbev danbev 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]>
@RaisinTen RaisinTen 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]>
@codebytere codebytere 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]>
@BethGriggs BethGriggs 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 free to 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