search

ukhas / habcloud-infra

HabCloud Infrastructure
1 stars 0 forks source link

make pep8 happy (also fix PEP3113 related sad) #4

Closed adamgreig closed 9 years ago

adamgreig commented 9 years ago

fixing PEP8 upsets in advance of adding new functions ;)

(in py3 you can't specify tuple arguments to lambdas or functions which is the PEP3113 part, not really sure if that's required fixing here but my linter is py3)

adamgreig commented 9 years ago

and p.s. I'm not convinced that https://github.com/ukhas/habcloud-infra/blob/pep8/vms/habcloud-vms#L215 actually needs the import?

danielrichman commented 9 years ago

by "will need it below" I mean that things we call will need it, so I want to fail now rather than later, if it is not present.

The idiom you use with "if BOTO" is perhaps better?