Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Multiple different ' ktlint' plugin references #135

Closed
lgvalle opened this issue Oct 5, 2018 · 4 comments
Closed

Multiple different ' ktlint' plugin references #135

lgvalle opened this issue Oct 5, 2018 · 4 comments
Labels
Milestone

Comments

@lgvalle
Copy link
Contributor

lgvalle commented Oct 5, 2018

I'm was going through the project documentation and code to setup a new ktlint integration and I found three different references to ktlint plugins:

Which one is the recommended one?
I believe only the first one is supported since is the one the configurator expects. If that the case we should ammend the documentation.

@mr-archano
Copy link
Contributor

mr-archano commented Oct 5, 2018

@lgvalle KtLint is a command line, standalone tool (no built-in Gradle support). The links in the docs are correct:

It appears though that the error message in KtlintConfigurator.groovy#L16 is mentioning the wrong Gradle plugin (it got uncaught since the introduction of support of KtLint in #110). It should be something easy to fix. Feel free to issue a PR :)

@tasomaniac
Copy link
Contributor

Thanks @mr-archano

I was looking at those 2 plugins at the time to see which one is more suitable. I think this was a copy paste mistake. :) The error message seems to be wrong.

@mr-archano
Copy link
Contributor

Fixed in #137

@lgvalle
Copy link
Contributor Author

lgvalle commented Oct 6, 2018

Thanks for the explanation & the fix @mr-archano 💯

@tasomaniac tasomaniac added this to the 0.7 milestone Nov 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants