You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe what should be investigated or refactored
There are several spots where Pepr calls process.exit(). This complicates the ability to do global logging, auditing, or cleanup. We should avoid this pattern to handle failure cases in a more graceful manner. Consider this explanation for why it's a bad idea.
We'll know we're done with this issue when each of those calls to process.exit() handle shutdown in a more graceful manner (e.g., throwing exceptions, passing control to some other function, etc.)
Describe what should be investigated or refactored
There are several spots where Pepr calls
process.exit()
. This complicates the ability to do global logging, auditing, or cleanup. We should avoid this pattern to handle failure cases in a more graceful manner. Consider this explanation for why it's a bad idea.We'll know we're done with this issue when each of those calls to
process.exit()
handle shutdown in a more graceful manner (e.g., throwing exceptions, passing control to some other function, etc.)Links to any relevant code
Additional context
Generate the above list with the command
grep "process.exit(" src/**/*.*s
The text was updated successfully, but these errors were encountered: