> @@ -1106,6 +1106,8 @@
>                <configLocation>resources/checkstyle.xml</configLocation>
>                <failOnViolation>true</failOnViolation>
>                <failsOnError>true</failsOnError>
> +              <!-- fails on itself as it uses the author tag in a module 
> definition -->
> +              <resourceExcludes>checkstyle.xml</resourceExcludes>

Other options here:

* do not add the exclude at this level, but override the plugin config in 
resources/pom.xml
* use a [suppressions 
filter](http://maven.apache.org/plugins/maven-checkstyle-plugin/examples/suppressions-filter.html)
 instead. That would narrow down the exception, but the line numbers would go 
out of sync pretty quickly as checkstyle.xml is updated, I suspect
* add [`<property name="fileExtension" 
value="java">`](http://checkstyle.sourceforge.net/config_regexp.html) or so to 
the `@author` module config. But we probably want to check all files (well, 
_except_ checkstyle.xml) for this tag

@andrewgaul Thoughts?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/537/files#r18384166

Reply via email to