[libvirt] [PATCH sandbox-image 0/9] Tweaks

Radostin Stoyanov posted 9 patches 6 years ago
Failed in applying to current master (apply log)
libvirt_sandbox_image/cli.py                 | 38 +++++++++++-----------------
libvirt_sandbox_image/sources/base.py        |  4 +--
libvirt_sandbox_image/sources/docker.py      | 10 ++++----
libvirt_sandbox_image/sources/virtbuilder.py |  4 +--
libvirt_sandbox_image/template.py            |  4 +--
scripts/virt-sandbox-image                   |  5 ++--
setup.py                                     | 17 ++++++-------
7 files changed, 37 insertions(+), 45 deletions(-)
mode change 100644 => 100755 setup.py
[libvirt] [PATCH sandbox-image 0/9] Tweaks
Posted by Radostin Stoyanov 6 years ago
This patch series introduces a few small changes towards making the code of
virt-sandbox-image compatible with PEP8.

Radostin Stoyanov (9):
  pylint: Remove unused import statements
  pylint: Move standard library imports on top
  pylint: Use consistent indentation of 4 spaces
  py3: Use 'builtins' instead of '__builtin__'
  pylint: Fix white-space around keywords
  setup: Add shebang
  cli: Remove unused constants
  cli: Remove redundant global statements
  docker: Add missing import base64

 libvirt_sandbox_image/cli.py                 | 38 +++++++++++-----------------
 libvirt_sandbox_image/sources/base.py        |  4 +--
 libvirt_sandbox_image/sources/docker.py      | 10 ++++----
 libvirt_sandbox_image/sources/virtbuilder.py |  4 +--
 libvirt_sandbox_image/template.py            |  4 +--
 scripts/virt-sandbox-image                   |  5 ++--
 setup.py                                     | 17 ++++++-------
 7 files changed, 37 insertions(+), 45 deletions(-)
 mode change 100644 => 100755 setup.py

-- 
2.14.3

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH sandbox-image 0/9] Tweaks
Posted by Daniel P. Berrangé 6 years ago
On Wed, Apr 18, 2018 at 09:57:38PM +0100, Radostin Stoyanov wrote:
> This patch series introduces a few small changes towards making the code of
> virt-sandbox-image compatible with PEP8.
> 
> Radostin Stoyanov (9):
>   pylint: Remove unused import statements
>   pylint: Move standard library imports on top
>   pylint: Use consistent indentation of 4 spaces
>   py3: Use 'builtins' instead of '__builtin__'
>   pylint: Fix white-space around keywords
>   setup: Add shebang
>   cli: Remove unused constants
>   cli: Remove redundant global statements
>   docker: Add missing import base64
> 
>  libvirt_sandbox_image/cli.py                 | 38 +++++++++++-----------------
>  libvirt_sandbox_image/sources/base.py        |  4 +--
>  libvirt_sandbox_image/sources/docker.py      | 10 ++++----
>  libvirt_sandbox_image/sources/virtbuilder.py |  4 +--
>  libvirt_sandbox_image/template.py            |  4 +--
>  scripts/virt-sandbox-image                   |  5 ++--
>  setup.py                                     | 17 ++++++-------
>  7 files changed, 37 insertions(+), 45 deletions(-)
>  mode change 100644 => 100755 setup.py

Thanks, I've pushed this series


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list