[PATCH 2/4] Fix some issues pylint found in the filter page.

David Shea dshea at redhat.com
Wed Jan 7 22:51:08 UTC 2015


On 01/07/2015 02:25 PM, Samantha N. Bueno wrote:
> Unused import, unused variable, attrs defined outside of __init__,
> and fixed a call to use the enlightbox.
> ---
>   pyanaconda/ui/gui/spokes/filter.py | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/pyanaconda/ui/gui/spokes/filter.py b/pyanaconda/ui/gui/spokes/filter.py
> index 19663ef..9e30075 100644
> --- a/pyanaconda/ui/gui/spokes/filter.py
> +++ b/pyanaconda/ui/gui/spokes/filter.py
> @@ -19,7 +19,7 @@
>   # Red Hat Author(s): Chris Lumens <clumens at redhat.com>
>   #
>   
> -# pylint: disable-msg=E0611
> +# pylint: disable=E0611
>   from gi.repository import Gtk

The pylint disable is an artifact of darker days and you can axe it. 
Also we can use symbolic names now (no-name-in-module) and it should be 
on the same line as the import to avoid applying to the entire file. For 
real though, kill it.


More information about the anaconda-patches mailing list