Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Exit zero if help/version is explicitly requested #525

Merged
merged 2 commits into from
Apr 8, 2024

Conversation

josh-berry
Copy link
Collaborator

@josh-berry josh-berry commented Apr 5, 2024

We want a non-zero exit when the user invokes temporal with an unknown command/subcommand, or when invoked with no arguments. But if the user explicitly asks for help or version information, we should print it and return success.

Closes #495.

We want a non-zero exit when the user invokes `temporal` with an unknown
command/subcommand, or when invoked with no arguments.  But if the user
explicitly asks for help, we should print it and return success.

Closes #495.
@josh-berry josh-berry requested a review from cretz April 5, 2024 20:43
Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

LGTM, but don't want to close #495 without also addressing --version or say we don't need to address it

@josh-berry
Copy link
Collaborator Author

LGTM, but don't want to close #495 without also addressing --version or say we don't need to address it

🤦 It even listed --version in the ticket, oops. Will fix and re-post.

@josh-berry josh-berry requested review from Sushisource and cretz April 5, 2024 23:10
@josh-berry josh-berry changed the title Exit zero if help is explicitly requested Exit zero if help/version is explicitly requested Apr 5, 2024
@josh-berry josh-berry merged commit 267e114 into cli-rewrite Apr 8, 2024
6 checks passed
@josh-berry josh-berry deleted the josh/exit-zero-on-help-495 branch April 8, 2024 16:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants