First Pull Request


Rule: Modifier order arturbosch/detekt

`private lateinit var` vs. `lateinit private var` Both are valid in Kotlin, but I'd like the ability to enforce one.

Created - 2 comments - easy feature formatting

Rule: SpreadOperator arturbosch/detekt

As mentioned in #155 and this [benchmark](https://sites.google.com/a/athaydes.com/renato-athaydes/posts/kotlinshiddencosts-benchmarks), using the spread operator can lead to performance costs comp...

Created - 3 comments - easy feature rules

Rule: Shadowed names arturbosch/detekt

The Kotlin compiler already prints a warning for each shadowed name. Should Detekt also detect these and report them? Example: ``` override fun bindItemView(position: Int, view: View, list: Li...

Created - 3 comments - easy feature hacktoberfest rules

Provide AddMutableFix for assignments with immutable references intellij-rust/intellij-rust

`RsAssignToImmutableInspection` [provides](https://github.com/intellij-rust/intellij-rust/pull/3200) `AddMutableFix` for values, but not for references. It would be useful to provide such fix for ...

Created - 3 comments - E-easy E-mentor code insight feature

Fail fast approach / configuration arturbosch/detekt

I'd like to have a fail fast approach where I'd say in the configuration file that I want every rule to be automatically turned on with the default values. That would also include rules that are ma...

Created - 5 comments - cli easy feature

Rule: Safe cast instead of if-else-null arturbosch/detekt

Instead of writing: ```kotlin if (element is KtElement) element else null ``` we can write ```kotlin element as? KtElement ```

Created - 5 comments - easy feature formatting

Rule TodoComment arturbosch/detekt

Basically flags the usage of `// TODO: something` Inspired by [this checkstyle rule](http://checkstyle.sourceforge.net/config_misc.html#TodoComment). Sidenote: I'd not flag `// TODO something...

Created - 7 comments - easy feature rules

Displaying all 7 issues

License

Language