Code review comment for lp:~maas-maintainers/juju-core/maas-provider-skeleton

Revision history for this message
William Reade (fwereade) wrote :

> > 8) StateInfo()
> >
> > can't this use instances()?
>
> Do you mean you'd prefer StateInfo() to use the internal method
> env.instances() instead of env.Instances()?

It was really just a question, there may be some detail that makes it inappropriate, but at first blush it seemed like the ideal candidate. Your judgment call.

« Back to merge proposal