summaryrefslogtreecommitdiff
path: root/test/command_callback/test_checkstyle_command_callback.vader
AgeCommit message (Collapse)Author
2019-11-27Fix 2913 - checkstyle config file ignored.Horacio Sanson
If checkstyle is configured with custom options that contain "-c" then the checkstyle config file option is ignored. This PR modifies the regular expression when creating the checkstyle command to avoid this.
2019-06-12Fix checkstyle default configuration.Horacio Sanson
Checkstyle xml configuration is mandatory and not providing one causes the tool to fail with the following error: Must specify a config XML file. Checkstyle itself contains a default configuration as part of its assests named `/google_checks.xml`. Invoking checkstyle with this config works even if such file does not exists in the file system: checkstyle -c /google_checks.xml This should be the default invocation to allow ALE to use checkstyle with zero configuration. Also when a user sets `g:ale_java_checkstyle_config` option, ALE should use it to invoke checkstyle even such file does not exists in the filesystem. This is because checkstyle is able to use configuration files within JAR files defined in the CLASSPATH. The default `/google_checks.xml` is an example of such configuration available within a JAR resource.
2019-06-03#2542 - Improve checkstyle project configurationw0rp