[virt-tools-list] [virt-bootstrap] [PATCH 3/7] pylint: Resolve test for membership

Cedric Bosdonnat cbosdonnat at suse.com
Thu Mar 1 09:26:13 UTC 2018


On Wed, 2018-02-28 at 00:11 +0000, Radostin Stoyanov wrote:
> E713 test for membership should be 'not in'
> 
> Signed-off-by: Radostin Stoyanov <rstoyanov1 at gmail.com>
> ---
>  src/virtBootstrap/sources/docker_source.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/virtBootstrap/sources/docker_source.py b/src/virtBootstrap/sources/docker_source.py
> index 0eace81..572018f 100644
> --- a/src/virtBootstrap/sources/docker_source.py
> +++ b/src/virtBootstrap/sources/docker_source.py
> @@ -96,7 +96,7 @@ class DockerSource(object):
>                                                  username=self.username,
>                                                  password=self.password)
>  
> -        if not 'Layers' in image_details or not image_details['Layers']:
> +        if 'Layers' not in image_details or not image_details['Layers']:
>              raise ValueError('No image layers.')
>  
>          # Layers are in order:

ACK

--
Cedric




More information about the virt-tools-list mailing list