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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return 0 code on help command and pass cli arguments to zig build run #45

Merged
merged 3 commits into from
Jan 26, 2022

Conversation

iddev5
Copy link
Contributor

@iddev5 iddev5 commented Jan 26, 2022

While working on #44 I noticed that

  1. zig build run -- --help is returning 1 while it should return 0 as its a successful execution of a command. After changing it, I found that,
  2. Build.zig is not passing CLI arguments to zig build run in the first place 😅

@marler8997
Copy link
Owner

marler8997 commented Jan 26, 2022

Looks fine, any idea why your PR doesn't trigger a run?

My guess is it has something to do with this line: https://github.com/marler8997/zigup/blob/master/.github/workflows/artifact.yml#L2

@iddev5
Copy link
Contributor Author

iddev5 commented Jan 26, 2022

Indeed that's it, it needs to be changed to on: [push, pull_request]

@marler8997
Copy link
Owner

Ok could you add a commit to this PR that changes that, then I can see the tests run and merge it.

@iddev5 iddev5 changed the title Return 0 code on helo command and pass cli arguments to zig build run Return 0 code on help command and pass cli arguments to zig build run Jan 26, 2022
@marler8997 marler8997 merged commit 5adde90 into marler8997:master Jan 26, 2022
@iddev5 iddev5 deleted the help-errcode branch January 26, 2022 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants