Re: [PATCH 02/10] build: change flake8 to use blacklist instead of whitelist

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

 



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é <berrange@xxxxxxxxxx>
---
  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
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux