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

26) forgot to mention this: env.Provider().(*maasEnvironProvider)

just use providerInstance?

27) var _ environs.EnvironProvider = (*maasEnvironProvider)(nil)

no fields, no pointer receivers -- you may as well assert that the value implements EnvironProvider, use the value everywhere, and just forget about the &s and *s.