Thanks @melanj! Just a couple comments:

* [ ] Address the comments above.
* [ ] Use a 3 space indent and a 120 line length in Java classes.
* [ ] Run `mvn checkstyle:checkstyle` from the `vcloud` folder and address the 
violations (you'll find them in the `target/checkstyle-result.xml` file).
* [ ] Add the unis tests for the `VCloudBasicAuthentication` filter.
* [ ] Add the unis tests for the `VCloudSupportedVersions` filter.

Could you take a look at this checklist? Thanks!

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

Reply via email to