Code review comment for lp:~gary/charms/oneiric/buildbot-slave/extra-packages

Revision history for this message
Benji York (benji) wrote :

This looks good.

A small redundancy: there's no need for the .strip() because the split
will remove any leading, trailing, or duplicate whitespace around the
package names.

review: Approve (code)

« Back to merge proposal