-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
feat: add option to use posix exit code upon fatal signal #4989
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
b419072
Option to use posix exit code upon fatal signal
73rhodes 7b36d1e
Update test/integration/options/posixExitCodes.spec.js
73rhodes 702ec53
Address PR review comments mocha org
73rhodes cb2ba84
Address PR review comments mocha org
73rhodes d000c32
Update docs/index.md
73rhodes 859abe6
Fix exit code when no signal caught
73rhodes 34e196d
Coverage for posix exit code with normally failing tests
73rhodes 7c85e3a
Update docs/index.md
73rhodes abdd3d5
Address PR comments; use os.constants
73rhodes e94876f
Remove test that asserts the problematic behavior
73rhodes 169927a
Omit win32 from signals test suite
73rhodes 73259a2
Update docs/index.md
73rhodes 5938f49
WIP - support posix-exit-codes option for both child-process and in-p…
73rhodes b155129
Update bin/mocha.js
73rhodes 5af8365
Update lib/cli/run-helpers.js
73rhodes d09c2bc
Update lib/cli/run-helpers.js
73rhodes 9c394f0
Update lib/cli/run-helpers.js
73rhodes bfe7388
Use preferred node import syntax
73rhodes b3bc5c5
Removed unused constants
73rhodes f365cbc
update run-helpers.js: Fix variable name
73rhodes f9ae3f5
Simplification of child process exit handlers
73rhodes 8b3f2e3
Add test case for numerical signal
73rhodes 7e0efd2
Update signal-handling tests; remove unsupported cases
73rhodes 55cbd8b
Merge pull request #13 from zendesk/dderidder/unit-tests
73rhodes 6613a02
Don't test for posix signals on windows
73rhodes 7347748
Update lib/cli/run-helpers.js
73rhodes File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Update docs/index.md
Co-authored-by: Josh Goldberg ✨ <[email protected]>
- Loading branch information
commit 7c85e3a0b0f28535e1cc91e47e0e8bb33f661fe6
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.