feat: allow custom exit command implementation

[axelrindle]

I'm creating a CLI which runs in an async context with init and dispose hooks. Just calling process.exit() when the user types exit may be fatal.

This change allows for custom implementation for an exit command.

[hongaar]

Thanks for creating this PR. I'm in favour of such a change.

I suggest a slightly different implementation though: make this option part of the global options passed to the Program constructor. Let's just call it exit to align better with the existing help and version options. What about it could take false or a function as value and defaults to () => process.exit(), would that work for your use case?

[axelrindle]

I suggest a slightly different implementation though...

Got it. Will implement this tomorrow.

What about it could take false or a function as value and defaults to () => process.exit(), would that work for your use case?

Yes. I will add that.

[axelrindle]

Ok, everything should be implemented now.

[hongaar]

Looks good to me, can you address the comment and add a description to the README? Then it's good to go. (If you can, a test for this would be appreciated)

[axelrindle]

I'm not familiar with jest, I suggest you add that test 😅

[hongaar]

Thanks for the PR!

[hongaar]

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀