Skip to content

src: remove unnecessary ToLocalChecked node_errors - #36547

Closed
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:node_errors_tolocalchecked
Closed

src: remove unnecessary ToLocalChecked node_errors#36547
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:node_errors_tolocalchecked

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 c++ Issues and PRs that require attention from people who are familiar with C++. label Dec 17, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@danbev

danbev commented Dec 22, 2020

Copy link
Copy Markdown
ContributorAuthor

Re-run of failing node-test-commit-osx ✔️
Re-run of failing node-test-commit-smartos ✔️

@danbevdanbev added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 22, 2020
danbev added a commit that referenced this pull request Dec 22, 2020
PR-URL: #36547
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danbev

Copy link
Copy Markdown
ContributorAuthor

Landed in fc8fcb0.

@danbevdanbev closed this Dec 22, 2020
@danbev
danbev deleted the node_errors_tolocalchecked branch December 22, 2020 08:35
targos pushed a commit that referenced this pull request Dec 22, 2020
PR-URL: #36547
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos pushed a commit that referenced this pull request May 1, 2021
PR-URL: #36547
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
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.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

@danbev@nodejs-github-bot@Trott@cjihrig@gireeshpunathil