Thanks @Ladicek!

>What is your preferred procedure for suggesting changes to the Azure ARM 
>provider? I understand that it's still heavily work in progress, but I wanted 
>to try it and provide feedback. And I thought that a PR would be the best 
>feedback :-)

PRs, design discussions and feedback are really great in this early stage or 
the provider :)

The code LGTM. I'd only change one thing: could you change the VMDeployment 
class to use AutoValue, similar to VMImage, VmHardware, etc? This was unnoticed 
before and now that you are adding fields to the class it would be a good time 
to fix that. Apart from that, @ritazh good to merge?

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/298#issuecomment-232476392

Reply via email to