amitdev / PMD-Intellij

Plugin for doing static analysis in Intellij using PMD
http://plugins.intellij.net/plugin/?id=1137
MIT License
71 stars 24 forks source link

Target JDK setting should be the supported newest by default and should be validated #146

Closed jborgers closed 6 months ago

jborgers commented 1 year ago

Target JDK is use by pmd tool for what java features and keywords to understand. So 17 should work for all supported versions (by pmd) of Java up to 17. Version used in IntelliJ for the project should be available, if that value is taken, I see no need to configure it.

jborgers commented 8 months ago

There may be a difference in what intelliJ uses and what is supported by the used PMD. PMD 6 only supports up to 20-preview. PMD 7 supports 21, however, it is not backwards compatible for custom rules.

jborgers commented 6 months ago

Should be 20-preview by default and be clear about it being the max supported version. And not accepting invalid versions.

jborgers commented 6 months ago

similar to #167

jborgers commented 6 months ago

works in 1.9.2