search

aspect-build / rules_lint

Run static analysis tools with Bazel
Apache License 2.0
76 stars 42 forks source link

[FR]: Kotlin linter #212

Closed smocherla-brex closed 5 months ago

smocherla-brex commented 5 months ago

What is the current behavior?

There's no support for Kotlin linting with rules_lint right now, but I thought'd be really nice if either ktlint (https://github.com/pinterest/ktlint) and/or detekt (https://github.com/detekt/detekt). While rules_kotlin has support for ktlint, it lacks many other configuration options like a custom ruleset jar for ktlint, and in general I really like the approach of rules_lint using aspects rather than their dedicated targets which we need to run as tests separately.

Describe the feature

Currently, no linter support is documented with Kotlin with rules_lint. We use Kotlin heavily at our company and it would be nice to plugin in rules_lint for this. I'm happy to spend cycles to put together PRs for this if it's something you'd be interested in accepting as a contribution.

alexeagle commented 5 months ago

Absolutely we'd like to take a PR for this, please let me know if contributor docs need work.

smocherla-brex commented 5 months ago

https://github.com/aspect-build/rules_lint/pull/215 is an attempt at adding support for ktlint.

alexeagle commented 5 months ago

Done in #215 thanks!