Thanks! Just some final minor comments about style and still some tests missing.
Also, note that the `checkNotNull` checks are redundant for parameters in 
constructors annotated with `@Inject`. Guice will already fail to isntantiate 
the class if they are null (unless declared `@Nullable`). So those checks could 
be removed.
Look forward to the final round of fixes so we can merge this!

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

Reply via email to