Force Push exit status
Closed this issue · 6 comments
ethanfrogers commented
I'm trying to utilize Force as an alternative to Ant for some internal automation. One thing I've run into is that push doesn't exit with the correct status code in the event of failures. I'm not sure if this is the desired behavior, but it makes automation a bit difficult since you can't fail based on exit code. Ideally, if there are any failures, force should exit with a non-zero status code. Would there be any interest in a PR for this or has this already been decided against?
dcarroll commented
Thanks for the issue post. A proper exit code is a great idea. It may
take a while to see that implemented in all commands. There is a command
ErrorAndExit method, but I don't think it is used everywhere. A PR would
be awesome!
…On Tue, Mar 7, 2017 at 1:00 PM, Ethan Rogers ***@***.***> wrote:
I'm trying to utilize Force as an alternative to Ant for some internal
automation. One thing I've run into is that push doesn't exit with the
correct status code in the event of failures. I'm not sure if this is the
desired behavior, but it makes automation a bit difficult since you can't
fail based on exit code. Ideally, if there are any failures, force should
exit with a non-zero status code. Would there be any interest in a PR for
this or has this already been decided against?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#380>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHGHv7YnlqX3MklDlkLJ1LgEVZND4oDks5rjcVxgaJpZM4MWBNN>
.
--
Thanks,
Dave
ethanfrogers commented
@cwarden that looks like exactly what I want. Any plans for submitting a PR for that?
cwarden commented
Just needed a reminder, I guess. Thanks, @ethanfrogers. I opened #381.
dcarroll commented
Yes, actually the ErrorAndExit does return a non-zero exit code. 👍