On 10/9/19 7:58 AM, Daniel P. Berrangé wrote:
The current flake8 check only looks at one item (semicolons at end of
line). This means that our code quality will continue to get worse,
violating an increasing number of checks.

Switching to a whitelist means that we freeze the badness at its
current level & can incrementally fix things up.

We are excluding the folllowing...

following


Indentation:


Warnings:

   W504 line break after binary operator
   W605 invalid escape sequence ‘x’

Might also be worth a mention that later patches will fix up some culprits and re-enable those checks.


Signed-off-by: Daniel P. Berrangé <[email protected]>
---
  build-aux/syntax-check.mk | 21 +++++++++++++++++++--
  1 file changed, 19 insertions(+), 2 deletions(-)
--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to