Code review comment for lp:~thumper/launchpad/recipe-new-ppa

Revision history for this message
Tim Penhey (thumper) wrote :

> I have one major complaint, though: It does not work.

Oops, must have not tested that bit.

I've updated it to make sure that the ppa name must be entered if a new ppa is being created, also default the ppa name to 'ppa' if it is the first ppa.

Updated the code also with Graham's suggestions.

« Back to merge proposal