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