Code review comment for lp:~postfuturist/mailman/rest-api-mods_owners

Revision history for this message
Barry Warsaw (barry) wrote :

Thanks for the code! It looks pretty good in general. I'm going to make a few stylistic fixes (indentation, parameter order, multiline import formatting), and I'll add a few more "bad path" tests to ensure the exception still gets raised when expected. Then I'll merge and commit.

Cheers!

review: Approve

« Back to merge proposal