search

appKODE / detekt-rules-compose

A collection of Detekt rules for Jetpack Compose
MIT License
136 stars 8 forks source link

Improve message on `ComposableEventParameterNaming` #2

Closed BraisGabin closed 2 years ago

BraisGabin commented 2 years ago

I'm getting legit errors like this:

/the/path/QuantityRow.kt:27:5: Invalid event parameter name. Use names like "onClick", "onValueChange" etc [ComposableEventParameterNaming]

It would be great to know what's the current parameter name from this message. Something like:

Invalid event parameter name `doSomething`. Use names like "onClick", "onValueChange", etc

If you agree I can open a PR changing it.

dimsuz commented 2 years ago

That would be great, thanks!