Re: [ovs-dev] [PATCH ovn] py-requirements: Remove hacking dependency and use recent flake8.

2023-11-13 Thread Ilya Maximets
On 11/13/23 17:23, Dumitru Ceara wrote: > On 11/13/23 15:15, Ilya Maximets wrote: >> On 11/13/23 14:47, Dumitru Ceara wrote: >>> On 11/13/23 14:41, Ilya Maximets wrote: On 11/13/23 14:33, Dumitru Ceara wrote: > Without this, when using Python 3.12 and flake8 5.0.4, the following > erro

Re: [ovs-dev] [PATCH ovn] py-requirements: Remove hacking dependency and use recent flake8.

2023-11-13 Thread Dumitru Ceara
On 11/13/23 15:15, Ilya Maximets wrote: > On 11/13/23 14:47, Dumitru Ceara wrote: >> On 11/13/23 14:41, Ilya Maximets wrote: >>> On 11/13/23 14:33, Dumitru Ceara wrote: Without this, when using Python 3.12 and flake8 5.0.4, the following errors are flagged: tests/check_acl_log.py:9

Re: [ovs-dev] [PATCH ovn] py-requirements: Remove hacking dependency and use recent flake8.

2023-11-13 Thread Ilya Maximets
On 11/13/23 14:47, Dumitru Ceara wrote: > On 11/13/23 14:41, Ilya Maximets wrote: >> On 11/13/23 14:33, Dumitru Ceara wrote: >>> Without this, when using Python 3.12 and flake8 5.0.4, the following >>> errors are flagged: >>> tests/check_acl_log.py:97:25: E231 missing whitespace after ':' >>> t

Re: [ovs-dev] [PATCH ovn] py-requirements: Remove hacking dependency and use recent flake8.

2023-11-13 Thread Dumitru Ceara
On 11/13/23 14:41, Ilya Maximets wrote: > On 11/13/23 14:33, Dumitru Ceara wrote: >> Without this, when using Python 3.12 and flake8 5.0.4, the following >> errors are flagged: >> tests/check_acl_log.py:97:25: E231 missing whitespace after ':' >> tests/check_acl_log.py:102:71: E231 missing whit

Re: [ovs-dev] [PATCH ovn] py-requirements: Remove hacking dependency and use recent flake8.

2023-11-13 Thread Ilya Maximets
On 11/13/23 14:33, Dumitru Ceara wrote: > Without this, when using Python 3.12 and flake8 5.0.4, the following > errors are flagged: > tests/check_acl_log.py:97:25: E231 missing whitespace after ':' > tests/check_acl_log.py:102:71: E231 missing whitespace after ':' > > This was reported and di

[ovs-dev] [PATCH ovn] py-requirements: Remove hacking dependency and use recent flake8.

2023-11-13 Thread Dumitru Ceara
Without this, when using Python 3.12 and flake8 5.0.4, the following errors are flagged: tests/check_acl_log.py:97:25: E231 missing whitespace after ':' tests/check_acl_log.py:102:71: E231 missing whitespace after ':' This was reported and discussed in a couple of contexts: https://mail.openvs