[Freeipa-devel] [PATCH] default values in host plugin

Rob Crittenden rcritten at redhat.com
Mon Apr 13 18:54:13 UTC 2009


Jason Gerard DeRose wrote:
> ack.
> 
> One Python style nitpick: you shouldn't import multiple packages/modules
> on the same line unless they're all from the same package (meaning
> you're using the "from" keyword).
> 
> So:
> 
>   import sys
>   import os
>   import platform
> 
> Instead of:
> 
>   import sys, os, platform
> 
> But this would be okay:
> 
>   from platform import architecture, system, uname
> 
> 
> 
> 
> 

pushed to master




More information about the Freeipa-devel mailing list