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.

Sign up for GitHub

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

Jump to bottom

Return correct exit codes #25

Closed
Blacksmoke16 opened this issue Aug 6, 2019 · 0 comments · Fixed by #26
Closed

Return correct exit codes #25

Blacksmoke16 opened this issue Aug 6, 2019 · 0 comments · Fixed by #26
Labels
kind:bug An existing feature isn't doing something correctly
Milestone
0.2.1

Comments

@Blacksmoke16
Copy link
Owner

Regression from https://github.com/Blacksmoke16/oq/releases/tag/v0.1.1. Failures from jq return a 0 exit code.

We need not do that if the process was not successful.
@Blacksmoke16 Blacksmoke16 added kind:bug An existing feature isn't doing something correctly status:wip labels Aug 6, 2019
@Blacksmoke16 Blacksmoke16 added this to the 0.2.1 milestone Aug 6, 2019
@Blacksmoke16 Blacksmoke16 mentioned this issue Aug 7, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind:bug An existing feature isn't doing something correctly
Projects
None yet
Milestone
0.2.1
Development

Successfully merging a pull request may close this issue.

Set correct exit code on jq failure Blacksmoke16/oq
1 participant
@Blacksmoke16