-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(#2122): add specific error status codes #2126
Conversation
This change adds specific exit status codes to identify different problems programatically
Hey, is there anything I can do to improve this PR to get it merged? :) |
Friendly ping :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I second the idea of having more specific result codes or error codes. This has already helped a lot in CI/CD scripting with other tools, so I believe this is also helpful with Bruno CLI.
Please consider my suggestions for changes that might further improve the proposed changes.
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
That's a good improvement. Thanks @andreassiegel! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, I like this!
@helloanoop Hey, did you have time to look at this change? Thanks! :) |
Anything? :) |
Friendly ping :) |
Is there anything I missed? Do I need to follow a different process to get a change merged? |
Anything @helloanoop ? |
Merged. Thank you @jzorn ! Thank you @andreassiegel @lohxt1 for the review. |
* fix: add specific error status codes usebruno#2122 This change adds specific exit status codes to identify different problems programatically * feat(usebruno#2122): Document the cli exit status codes * Update packages/bruno-cli/src/constants.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/constants.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> * Update packages/bruno-cli/src/commands/run.js Co-authored-by: Andreas Siegel <mail@andreassiegel.de> --------- Co-authored-by: Andreas Siegel <mail@andreassiegel.de>
Description
This change adds specific exit status codes to identify different problems programatically, fixes #2122
Contribution Checklist:
Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.