Skip to content

Conversation

@danbev
Copy link
Contributor

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

@nodejs-github-botnodejs-github-bot added the tools Issues and PRs related to the tools directory. label Nov 24, 2020
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@danbev
Copy link
ContributorAuthor

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

@danbevdanbev added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 25, 2020
danbev added a commit that referenced this pull request Nov 27, 2020
PR-URL: #36245 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]>
@danbev
Copy link
ContributorAuthor

Landed in 0a23d65.

@danbevdanbev closed this Nov 27, 2020
@danbevdanbev deleted the snapshot_builder_using branch November 27, 2020 04:55
danielleadams pushed a commit that referenced this pull request Dec 7, 2020
PR-URL: #36245 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]>
@danielleadamsdanielleadams mentioned this pull request Dec 7, 2020
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.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@danbev@nodejs-github-bot@jasnell@lpinca@cjihrig@richardlau@gireeshpunathil@targos