-
Notifications
You must be signed in to change notification settings - Fork 17
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
Maven plugin complaining about trailing commas when those checks are disabled #507
Comments
If it's not noise, here is what
|
I rechecked, and the only |
I have the same issue. It seems that version 1.15.0 ignores |
|
Apologies if this is covered elsewhere. I checked https://github.com/gantsign/ktlint-maven-plugin and didn't see anything on the front page on changes in this area. @freemanjp Lots of warnings that I should use And version 1.15.1 is just what I was looking for. It unblocks me on multiple Kotlin projects for upgrading the plugin. |
tl;dr -- The Maven plugin mishandles trailing comma settings
Given this
.editorconfig
in my Maven project root:When I run Ktlint command line tool as
ktlint 'src/**/*.kt'
(version 0.47.0, latest as of this post), it produces no complaints, andktlint -F 'src/**/*.kt'
produces no file changes.When I run with the Ktlint Maven plugin, the report or check goals complain about trailing commas both for call and declaration sites.
The
pom.xml
runs the plugin like this ("ktlint-maven-plugin.version" is set to 1.15.0):I was previously able to disable other rules using
.editorconfig
(such as "import-ordering" to work around IntelliJ being non-standard -- the irony).The text was updated successfully, but these errors were encountered: