[MCHECKSTYLE-366] Upgrade checkstyle to a more recent version

• Type: Dependency upgrade
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0
• Fix Version/s: 3.1.0
• Assignee: Enrico Olivelli
• Reporter: Olivier Grégoire
• Created: 22/Jan/19 16:58
• Updated: 23/Dec/20 09:01
• Resolved: 22/Apr/19 22:23

Description

Version 3.0.0 still uses checkstyle 6.18.

 

The google_checks.xml of 6.18 contains a rule that says "com.google" imports must be first. This was removed in 2016 yet the latest version of checkstyle (3.0.0 of January 2018) still says I should use that version of 2016 which has the dummy rule that com.google imports should come first.

 

I know it's overridable, but the bottom line is that checkstyle relies on an really outdated version of checkstyle which has total nonsense in it.

Issue Links

• is duplicated by: MCHECKSTYLE-323 usage of checkstyle 7.0 (jdk8 is required)
• links to: GitHub Pull Request #13