search

jfmengels / node-elm-review

CLI for elm-review
https://package.elm-lang.org/packages/jfmengels/elm-review/latest/
BSD 3-Clause "New" or "Revised" License
48 stars 25 forks source link

fix process.exit calls #151

Closed BrianHicks closed 6 months ago

BrianHicks commented 6 months ago

This replaces all instances of process.exit with a new vendored dependency exit that drains all streams before exiting for real.

Fixes the issue we talked about in the Elm slack. For when that gets eaten by the Slack history monster, the problem is that process.exit would cut off JSON output when run from within a Node parent process.

jfmengels commented 6 months ago

Seems to work really well. Thank you very much :heart: :pray: