Re: [virt-manager PATCH] pylint: disable undefined-loop-variable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/14/2017 10:17 PM, Chen Hanxiao wrote:
> From: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
> 
>   pylint brings noise, disable it.
> 
> Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
> ---
>  virtinst/urlfetcher.py | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/virtinst/urlfetcher.py b/virtinst/urlfetcher.py
> index 4abc83a..b5b1244 100644
> --- a/virtinst/urlfetcher.py
> +++ b/virtinst/urlfetcher.py
> @@ -500,6 +500,7 @@ def getDistroStore(guest, fetcher):
>                  found = True
>                  break
>  
> +        # pylint: disable=undefined-loop-variable
>          if found:
>              logging.debug("Prioritizing distro store=%s", store)
>              stores.remove(store)
> 

Thanks for the patch, but I fixed this in a different way (changed the way the
loop works), along with fixing a bunch more pylint warnings with latest pylint

- Cole

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list



[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux