search for: e251

...@@ -34,12 +34,9 @@ done # Checks the files correspond to PEP8 coding style. # https://www.python.org/dev/peps/pep-0008/ -if $python-pep8 --version >/dev/null 2>&1; then - for f in $files; do - # Ignore: - # E226 missing whitespace around arithmetic operator - # E251 unexpected spaces around keyword / parameter equals - # E302 expected 2 blank lines, found 1 - $python-pep8 --ignore=E226,E251,E302 "$f" - done +if test "x$PYCODESTYLE" != xno; then + # Ignore: + # E501 line too long + # E722 do not use bare 'except...

Fixes the majority of the pycodestyle issues in the Python scripts, and fix the existing test-v2v-python-syntax.sh to use pycodestyle to actually perform style checks. Pino Toscano (6): PEP 8: adapt whitespaces in lines PEP 8: move imports at the top PEP 8: adapt empty lines tests: find all the Python scripts for syntax checks -o rhv-upload: remove unused Python imports Revamp check

Fix one dist issue, and almost all the builddir!=srcdir issues. (For the record, only 3 tests still fail in that setup.) Pino Toscano (3): libvirt-ocaml: add libvirt_c.h as source tests: fix path to sources of fake-virtio-win.iso tests: fix srcdir references bundled/libvirt-ocaml/Makefile.am | 1 + test-data/fake-virtio-win/Makefile.am | 2 +- tests/rhbz1232192.sh

v7 was here: https://www.redhat.com/archives/libguestfs/2018-March/msg00143.html Since then: - Earlier patches are now upstream. - The to-do list is moved from the commit message to the TODO file. - This version forces -of raw + -oa sparse and gives an error in any other mode. We intend to lift these restrictions later. - Tested against latest imageio which supports longer timeouts,

...-m py_compile $f +done + +# Checks the files correspond to PEP8 coding style. +# https://www.python.org/dev/peps/pep-0008/ +if $python-pep8 --version >/dev/null 2>&1; then + for f in $files; do + # Ignore: + # E226 missing whitespace around arithmetic operator + # E251 unexpected spaces around keyword / parameter equals + # E302 expected 2 blank lines, found 1 + $python-pep8 --ignore=E226,E251,E302 $f + done +fi diff --git a/v2v/virt-v2v.pod b/v2v/virt-v2v.pod index 5102e6841..4eab5a55d 100644 --- a/v2v/virt-v2v.pod +++ b/v2v/virt-v2v.pod @@ -6,1...

...-m py_compile $f +done + +# Checks the files correspond to PEP8 coding style. +# https://www.python.org/dev/peps/pep-0008/ +if $python-pep8 --version >/dev/null 2>&1; then + for f in $files; do + # Ignore: + # E226 missing whitespace around arithmetic operator + # E251 unexpected spaces around keyword / parameter equals + # E302 expected 2 blank lines, found 1 + $python-pep8 --ignore=E226,E251,E302 $f + done +fi diff --git a/v2v/virt-v2v.pod b/v2v/virt-v2v.pod index 5102e6841..8f4ae02c5 100644 --- a/v2v/virt-v2v.pod +++ b/v2v/virt-v2v.pod @@ -6,1...

v8 was here: https://www.redhat.com/archives/libguestfs/2018-April/msg00022.html v8 -> v9: - Addresses the only feedback from Tomáš. Rich.