Skip to content

src: use basename(argv0) for --trace-uncaught suggestion - #32798

Closed
addaleax wants to merge 2 commits into
nodejs:masterfrom
addaleax:uncaught-exception-basename
Closed

src: use basename(argv0) for --trace-uncaught suggestion#32798
addaleax wants to merge 2 commits into
nodejs:masterfrom
addaleax:uncaught-exception-basename

Conversation

@addaleax

Copy link
Copy Markdown
Member

Refs: #32797 (comment)

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

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. fs Issues and PRs related to file-system APIs and the fs module. labels Apr 12, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Comment thread src/node_file.cc Outdated
@addaleax addaleax added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Apr 12, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

nodejs-github-bot commented Apr 13, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/30690/ (:white_check_mark:)

Comment thread src/node_errors.cc
addaleax added a commit that referenced this pull request Apr 15, 2020
Refs: #32797 (comment)

PR-URL: #32798
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@addaleax

Copy link
Copy Markdown
Member Author

Landed in e0a7fd7

@addaleax addaleax closed this Apr 15, 2020
@addaleax
addaleax deleted the uncaught-exception-basename branch April 15, 2020 00:11
MylesBorins pushed a commit that referenced this pull request Apr 17, 2020
Refs: #32797 (comment)

PR-URL: #32798
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@targos targos added baking-for-lts PRs whose changes need more time before an LTS backport. lts-watch-v12.x and removed author ready PRs with CI started, the required approvals, and no outstanding review comments. labels Apr 25, 2020
BridgeAR pushed a commit that referenced this pull request Apr 28, 2020
Refs: #32797 (comment)

PR-URL: #32798
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@BridgeAR BridgeAR mentioned this pull request Apr 28, 2020
codebytere pushed a commit that referenced this pull request Jun 7, 2020
Refs: #32797 (comment)

PR-URL: #32798
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@codebytere codebytere mentioned this pull request Jun 9, 2020
codebytere added a commit to electron/electron that referenced this pull request Jun 18, 2020
@targos targos removed the baking-for-lts PRs whose changes need more time before an LTS backport. label Sep 16, 2022
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++. fs Issues and PRs related to file-system APIs and the fs module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants