> @@ -384,7 +400,7 @@ public static DockerTemplateOptions 
> overrideLoginCredentials(LoginCredentials cr
>        }
>  
>        /**
> -       * @see TemplateOptions#blockUntilRunning
> +       * @see TemplateOptions#blockUntilRunning(boolean)
>         */
>        public static DockerTemplateOptions blockUntilRunning(boolean 
> blockUntilRunning) {
>           DockerTemplateOptions options = new DockerTemplateOptions();

A quick look to all existing template options subclasses show that the only 
ones that follow this pattern are the ones for Docker, SoftLayer and Azure 
Compute.
Given that this is not the common pattern and that this PR already cleans up 
the options in Docker, I think it's better to use this PR to remove the 
optionals from Docker, and fix SoftLayer and Azure in different PRs.

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

Reply via email to