Consider honoring `PrivatePropertyName` like `PropertyName`

pinterest / ktlint

An anti-bikeshedding Kotlin linter with built-in formatter

https://pinterest.github.io/ktlint/

MIT License

6.07k stars 504 forks source link

Closed Goooler closed 2 months ago

Goooler commented 3 months ago

PrivatePropertyName is from IntelliJ, we can honor it like PropertyName, links #2441.

Error throwing:

Property name should start with a lowercase letter and use camel case (standard:property-naming)

paul-dingemans commented 3 months ago

We can do so. But according to the inspection settings there even seem to be more inspections/suppression that can be used as a trigger.

We have to create code samples that trigger those warnings to find the id of the suppression. We also need these for the unit tests. Alternatively, we can look them up. For example see https://github.com/JakeWharton/kotlin/blob/4383335168338df9bbbe2a63cb213a68d0858104/idea/src/META-INF/plugin.xml.173#L2546