Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

process: Add exitCode property#6190

Merged
isaacs merged 2 commits intonodejs:masterfrom
isaacs:process-exit-code
Sep 7, 2013
Merged

process: Add exitCode property#6190
isaacs merged 2 commits intonodejs:masterfrom
isaacs:process-exit-code

Conversation

@isaacs
Copy link
Copy Markdown

@isaacs isaacs commented Sep 6, 2013

This allows one to set a specific status code, while still letting the process exit gracefully once all async operations are completed.

This allows one to set a specific status code, while still letting the
process exit gracefully once all async operations are completed.
@othiym23
Copy link
Copy Markdown

othiym23 commented Sep 6, 2013

Useful! Cool! :shipit:

This should always be used in the case of an uncaughtException
@isaacs isaacs merged commit 6808706 into nodejs:master Sep 7, 2013
Comment thread src/node.cc
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/Handle/Local/ and s/exitCode/exit_code/ (the C++ code base uses snake case (almost) everywhere.) exit_code_string would be even better.

defunctzombie pushed a commit to defunctzombie/npm that referenced this pull request Nov 5, 2013
As a polyfill to the node process.exitCode behavior - see
nodejs/node-v0.x-archive#6190
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants