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

make successful CLI execution silent #22

Open
mhuisman opened this issue Feb 28, 2017 · 0 comments
Open

make successful CLI execution silent #22

mhuisman opened this issue Feb 28, 2017 · 0 comments

Comments

@mhuisman
Copy link

mhuisman commented Feb 28, 2017

in reporter.js,

similar to https://github.com/zallek/swagger-diff/blob/master/src/reporter.js#L29 and https://github.com/zallek/swagger-diff/blob/master/src/reporter.js#L36, move https://github.com/zallek/swagger-diff/blob/master/src/reporter.js#L22 and https://github.com/zallek/swagger-diff/blob/master/src/reporter.js#L15 within their respective if-blocks.

CLI users could then (given the correct config rules) interpret non-empty stdout as evidence of diff failure.

I'm happy to put together a PR if that would help. Thanks in advance!

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

No branches or pull requests

1 participant