Sure, I fixed and verified with “make flake8-check”. Ted > On Jan 16, 2019, at 12:29 PM, Ben Pfaff <[email protected]> wrote: > > On Fri, Jan 04, 2019 at 02:16:36AM +0000, Ted Elhourani wrote: >> The Python IDL implementation supports ovsdb cluster connections. >> This patch is a follow up to commit 31e434fc98, it adds the option of >> connecting to the leader (the default) in the Raft-based cluster. It mimics >> the exisiting C IDL support for clusters introduced in commit 1b1d2e6daa. > > I'm getting the following reports from flake8 when I apply this patch. > Can you fix those up for v3? > > ../python/ovs/db/idl.py:43:1: E302 expected 2 blank lines, found 1 > ../python/ovs/db/idl.py:101:80: E501 line too long (85 > 79 characters) > ../python/ovs/db/idl.py:125:80: E501 line too long (81 > 79 characters) > ../python/ovs/db/idl.py:579:9: F841 local variable 'res' is assigned to but > never used > ../python/ovs/db/idl.py:748:80: E501 line too long (84 > 79 characters) > > Thanks, > > Ben.
_______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
