Code review comment for lp://qastaging/~sergiusens/snapcraft/localplugs--
Revision history for this message
| Sergio Schvezov (sergiusens) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
demo site
(Get the code!)
On Tue, Sep 22, 2015 at 1:34 AM, Leo Arias <email address hidden> wrote:
> oh, and I think you don't need this:
>
> + module = None
> + if not module:
>
> Just assign the module on the except ImportError: block.
>
I had that initially, I just didn't like the Exception within Exception
error so I kept the handler at a minimum ;-)