WildcardImport should support regex exclusions

[rock3r]

Currently WildcardImport only checks if a given wildcard import matches exactly entries from a set of exclusions, but it should support regexes because a lot of people have wildcard imports for stuff like Kotlin Android Extension's kotlinx.android.synthetic.* packages, which are very easy to allow with a regex but almost impossible to exclude without.

Having kotlinx.android.synthetic use star imports is a default IDE configuration so we should likely allow those by default too.

[arturbosch]

Agree, we also have the logic for it see LateInitUsage.excludeAnnotatedProperties and the Excludes class.

[arturbosch]

Just checked out. It is already implemented. The config property was just absent ;)
Edit: Having a list of default annotation patterns seems fine for me. Are there more common patterns in Android that should be excluded by default?

[rock3r]

Hmmm not that I know of. That was a glaring example as it's a default one in Android Studio (and I suppose IDEA), and Android Kotlin projects get the Android Kotlin extensions by default in the template...

[rock3r]

So these are the ones the IDE allows and converts to wildcards by default:

I suppose we should have those as defaults

[arturbosch]

I've added intellij's defaults to the default config ^^

[rock3r]

Great, thanks :)

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related topics.