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

Non-zero exit status #14

Closed
jayvdb opened this issue Dec 18, 2017 · 3 comments
Closed

Non-zero exit status #14

jayvdb opened this issue Dec 18, 2017 · 3 comments

Comments

@jayvdb
Copy link
Member

jayvdb commented Dec 18, 2017

If there were any modifications, it would be helpful if there was a non-zero status.

This would allow moban to be used in CI to detect (and usually fail) if all files were not in sync.

This could be done like git diff --exit-code which adds the exit code to reflect there were changes.

@chfw
Copy link
Member

chfw commented Dec 18, 2017

what about returning the number of templated files? 0 for no files, 1 for 1 files... n for N files.

@chfw chfw closed this as completed in b631580 Dec 18, 2017
@chfw
Copy link
Member

chfw commented Dec 18, 2017

1 for having and acted on changes, 2 for errors.

Just checked that N is impossible as the upper limit is 255.

chfw added a commit that referenced this issue Dec 19, 2017
@chfw
Copy link
Member

chfw commented Dec 19, 2017

please be notified that I have renamed setupmobans to pypi-mobans.

chfw added a commit that referenced this issue Jan 11, 2019
✨ update exit code. resolves #157. update #14.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants