ccaominh / intellij-inspect

Docker container for running IntelliJ IDEA code inspections.
Apache License 2.0
2 stars 0 forks source link

Update IntelliJ version #8

Open ccaominh opened 4 years ago

ccaominh commented 4 years ago

Currently IntelliJ 2018.3.6 is used as running the inspection analysis on https://github.com/apache/druid/ with IntelliJ 2019.3.2 causes IntelliJ to throw a NullPointerException during the unused declaration analysis.

This issue is being tracked by https://youtrack.jetbrains.com/issue/IDEA-232022 and is possibly fixed in IntelliJ 2020.1: https://youtrack.jetbrains.com/issue/IDEA-227476#focus=streamItem-27-3892422.0-0

ambition-consulting commented 3 years ago

Any update on this? Would be great to get the new IDE included in our CI pipeline.

Congratulations by the way for this great tool - it's a life saver in an environment with hundreds of developers, where reviewing must be automated as much as possible!

ccaominh commented 3 years ago

I haven't looked into this for a while. I'll try out the latest version of IntelliJ and see how it fares.

ambition-consulting commented 3 years ago

Awesome, thank you!

Chi Cao Minh @.***> schrieb am Do., 29. Apr. 2021, 00:46:

I haven't looked into this for a while. I'll try out the latest version of IntelliJ and see how it fares.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ccaominh/intellij-inspect/issues/8#issuecomment-828829759, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAF3XTMXURATO2WJ3KTQ23TLCF2ZANCNFSM4KO4WGFA .

ambition-consulting commented 3 years ago

Any progress? I am getting some false positives on the inspections, that I hope to get rid off.

ccaominh commented 3 years ago

I tried IntelliJ 2020 and 2021 but still get errors on https://github.com/apache/druid/ during the unused declaration analysis.

ccaominh commented 2 years ago

The NPE still occurs when using IntelliJ 2021.3.1 to run inspections on recent Apache Druid (7 Jan 2022 -- https://github.com/apache/druid/commit/7c17341caa7d8255389128eb99f24863593abcbd). Left a comment on the YouTrack ticket too: https://youtrack.jetbrains.com/issue/IDEA-232022#focus=Comments-27-5703328.0-0