Code review comment for lp:~johnsca/charm-helpers/adduser-group

Revision history for this message
Cory Johns (johnsca) wrote :

Thanks for the feedback. I added a test and docstring, changed the param names as suggested, and made it always include the default primary group to avoid errors like this: http://pastebin.ubuntu.com/13506551/

« Back to merge proposal