Code review comment for lp:~newell-jensen/maas/bulk-node-actions-messages-bug-1387968

Revision history for this message
Newell Jensen (newell-jensen) wrote :

No new separate test but I did update a test that was already there for the
functionality that I introduced.

On Mon, Nov 3, 2014 at 10:44 AM, Blake Rouse <email address hidden>
wrote:

> Review: Needs Fixing
>
> No tests?
>
> Also the commit message is a little long. I would give exactly what this
> does or fixes. Explaining the better way in the description.
> --
>
> https://code.launchpad.net/~newell-jensen/maas/bulk-node-actions-messages-bug-1387968/+merge/240217
> You are the owner of
> lp:~newell-jensen/maas/bulk-node-actions-messages-bug-1387968.
>

« Back to merge proposal