Skip to content

Conversation

@BridgeAR
Copy link
Member

This makes sure process._fatalException() returns a boolean when
run inside of a worker.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

This makes sure `process._fatalException()` returns a boolean when run inside of a worker.
@nodejs-github-bot
Copy link
Collaborator

@BridgeARBridgeAR added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. worker Issues and PRs related to Worker support. labels Sep 25, 2019
@nodejs-github-bot
Copy link
Collaborator

@Trott
Copy link
Member

Landed in aea9a0f

@TrottTrott closed this Sep 27, 2019
Trott pushed a commit that referenced this pull request Sep 27, 2019
This makes sure `process._fatalException()` returns a boolean when run inside of a worker. PR-URL: #29706 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>
targos pushed a commit that referenced this pull request Oct 1, 2019
This makes sure `process._fatalException()` returns a boolean when run inside of a worker. PR-URL: #29706 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>
@targostargos mentioned this pull request Oct 1, 2019
@BridgeARBridgeAR deleted the fix-fatal-exception-return-type branch January 20, 2020 12:05
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.workerIssues and PRs related to Worker support.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@BridgeAR@nodejs-github-bot@Trott@jasnell@addaleax@lpinca@devnexen@JungMinu