Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 correct error and exit status handling #32

Closed
jeftadlvw opened this issue May 27, 2024 · 0 comments
Closed

馃悰 correct error and exit status handling #32

jeftadlvw opened this issue May 27, 2024 · 0 comments
Labels
architecture-improvement Improves the overall architecture bug::comfirmed Something isn't working severity::okay This issue needs to be addressed within near time
Milestone

Comments

@jeftadlvw
Copy link
Owner

Currently, command handling functions don't return an error, and no actual errors are handled at the root command. Thus, git-nest returns an exit status of 0, which is false behaviour when errors do occur.

Cobra has this functionality built-in:

I see no reason to not utilize RunE instead of Run.

@jeftadlvw jeftadlvw added bug::comfirmed Something isn't working severity::okay This issue needs to be addressed within near time architecture-improvement Improves the overall architecture labels May 27, 2024
@jeftadlvw jeftadlvw added this to the v0.0.2 milestone May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architecture-improvement Improves the overall architecture bug::comfirmed Something isn't working severity::okay This issue needs to be addressed within near time
Projects
None yet
Development

No branches or pull requests

1 participant