launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #15008
Re: [Merge] lp:~rvb/maas/gomaasapi-listnodes into lp:~maas-maintainers/maas/gomaasapi
Okay, you've sold me: I'll import gocheck using "." from now on.
As for the long test setup, there's a lot of server/client setup that looks repetitive. It's not particularly tightly coupled, so a helper to make a client, server, and single-serving http server may seem a bit arbitrary. But maybe you won't need all 3 as return values that way, and in that case I think it would be justifiable.
Jeroen
--
https://code.launchpad.net/~rvb/maas/gomaasapi-listnodes/+merge/144654
Your team MAAS Maintainers is subscribed to branch lp:~maas-maintainers/maas/gomaasapi.
References