Skip to content

tools: use using-declaration consistently - #36245

Closed
danbev wants to merge 2 commits into
nodejs:masterfrom
danbev:snapshot_builder_using
Closed

tools: use using-declaration consistently#36245
danbev wants to merge 2 commits into
nodejs:masterfrom
danbev:snapshot_builder_using

Conversation

@danbev

Copy link
Copy Markdown
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
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@danbev

Copy link
Copy Markdown
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
Comment threadtools/snapshot/snapshot_builder.cc Outdated
danbev added a commit that referenced this pull request Nov 27, 2020
PR-URL: #36245
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@danbev

Copy link
Copy Markdown
ContributorAuthor

Landed in 0a23d65.

@danbevdanbev closed this Nov 27, 2020
@danbev
danbev 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 <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@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