Re: [PATCH 4/5] python: add qapi static analysis tests

2025-03-26 Thread John Snow
On Wed, Mar 26, 2025 at 2:13 AM Markus Armbruster wrote: > John Snow writes: > > > On Tue, Mar 25, 2025 at 3:53 AM Markus Armbruster > wrote: > > > >> John Snow writes: > >> > >> > Update the python tests to also check qapi. No idea why I didn't do > this > >> > before. I guess I was counting

Re: [PATCH 4/5] python: add qapi static analysis tests

2025-03-25 Thread Markus Armbruster
John Snow writes: > On Tue, Mar 25, 2025 at 3:53 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > Update the python tests to also check qapi. No idea why I didn't do this >> > before. I guess I was counting on moving it under python/ and then just >> > forgot after that was NACKed. Oo

Re: [PATCH 4/5] python: add qapi static analysis tests

2025-03-25 Thread John Snow
On Tue, Mar 25, 2025 at 3:53 AM Markus Armbruster wrote: > John Snow writes: > > > Update the python tests to also check qapi. No idea why I didn't do this > > before. I guess I was counting on moving it under python/ and then just > > forgot after that was NACKed. Oops, this turns out to be rea

Re: [PATCH 4/5] python: add qapi static analysis tests

2025-03-25 Thread Markus Armbruster
John Snow writes: > Update the python tests to also check qapi. No idea why I didn't do this > before. I guess I was counting on moving it under python/ and then just > forgot after that was NACKed. Oops, this turns out to be really easy. > > flake8, isort and mypy use the tool configuration from

[PATCH 4/5] python: add qapi static analysis tests

2025-03-21 Thread John Snow
Update the python tests to also check qapi. No idea why I didn't do this before. I guess I was counting on moving it under python/ and then just forgot after that was NACKed. Oops, this turns out to be really easy. flake8, isort and mypy use the tool configuration from the existing python director