-
Notifications
You must be signed in to change notification settings - Fork 509
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
Allow Unit in .editorconfig #313
Comments
does ktlint fails or warns ? the idea of ktlint in the generated code, is to reformat the code to be more kotlin-ish. |
ktlint can do both, the only problem is that it will remove |
Ability to disable rules on per-file basis is coming in #208. |
That's great news, thanks @shyiko!! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, looking at the Jetbrains coding conventions, for libraries is suggested to always explicitly specify the return type of a function, even if it's
Unit
. However, as far as I know, ktlint doesn't allow to disable the check on uselessUnit
globably. Do you think it make sense to have it as an option i the.editorconfig
file? And if not, what would be your suggestion to disable theUnit
check globably for a library project?The text was updated successfully, but these errors were encountered: