This looks good, but the checkstyle configuration should be removed. The 
[config is taken from the 
classpath](https://github.com/jclouds/jclouds/blob/master/project/pom.xml#L675-L676)
 (to reuse them in all jclouds related projects), so once 
https://github.com/jclouds/jclouds/pull/193 is merged, the RedundantModifier 
confioguration should be available for jclouds-chef too.

The PR should only contain the chanes in the code, but not the configuration.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/35#issuecomment-27472238

Reply via email to