[MCHECKSTYLE-307] Upgrade to Checkstyle 6.11

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.16
• Fix Version/s: 2.17
• Assignee: Michael Osipov
• Reporter: Michael Osipov
• Created: 02/Sep/15 07:41
• Updated: 29/Dec/15 00:28
• Resolved: 30/Sep/15 19:48

Description

Hi folks,

I have upraded Checkstyle locally and almost all ITs pass except for the issues [e-mail] already has described:

Changes in Checkstyle 6.3 that impact this plugin:

• The IT MCHECKSTYLE-214-basedir-resource fails after upgrade to Checkstyle 6.3 with NPE - I have not found a solution for this yet

Changes in Checkstyle 6.6 that impact this plugin:

• DefaultLogger( OutputStream, boolean ) now throws UnsupportedEncodingException - this is easy to fix

The first issue is caused because the default checkfiles apply to specific file extensions now, so IT for MCHECKSTYLE-214. rfscholte, what do you think about it. We can invert the test or add a custom sun_checks.xml.
The second one can be fixed with the linked GH issue.

I am willing to promote a release if I get some help to resolve the issues above.

Issue Links

• is depended upon by: MCHECKSTYLE-310 Parrallel build failing with various errors
• is depended upon by: MCHECKSTYLE-306 Support "ignoreInlineTags" property introduced in Checkstyle 6.8
• is related to: MCHECKSTYLE-252 Version 2.13 includes non-source files in checks
• supercedes: MCHECKSTYLE-292 Upgrade to Checkstyle 6.6
• supercedes: MCHECKSTYLE-298 Upgrade to Checkstyle 6.7
• supercedes: MCHECKSTYLE-300 Upgrade to Checkstyle 6.8.x+
• links to: GitHub issue on exception