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

I don't think instances() would make a good replacement for Instances(). It's basically the same thing, except instances() has a quirk to its API that made it suit the implementation of multiple exported functions. Instances() says what we want more precisely and with less risk of accident.