-
Notifications
You must be signed in to change notification settings - Fork 503
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
Why ktlint doesn't fails when auto-correct is enabled #162
Comments
Good catch! Fixed in 0.17.0 (Maven Central sync might take up to 25 minutes). |
Thank you! too fast! |
Hi, I have the same issue that was reported before the fix. When I run ktlint -F the commit passes even though there are reported formatting issues which cannot be auto corrected. I would like the commit to be failed in that case. In the docs for ktlint -F it says: "auto-correct style violations (if some errors cannot be fixed automatically they will be printed to stderr)". Which doesn't mention that failing the commit is intended. Is it possible to have the commit fail if there are issues which cannot be auto corrected? Thanks! |
When I run
ktlint -F
it doesn't fail when there is something that ktlint can't resolve it, only show error on the screen but doesn't return 1.The text was updated successfully, but these errors were encountered: