Re: [PATCH 2/6] test: Add linting of Python test scripts

[Stefano Brivio <sbrivio@redhat.com>]

On Wed, 1 Oct 2025 12:40:09 +0200
Paul Holzinger <pholzing@redhat.com> wrote:

> On 01/10/2025 12:23, Stefano Brivio wrote:
> > On Wed,  1 Oct 2025 19:51:58 +1000
> > David Gibson <david@gibson.dropbear.id.au> wrote:
> >  
> >> We currently have one test moved to the new exeter based framwork written
> >> in Python.  We plan to add many more, so add linting (flake8) and type
> >> checking (mypy) of those scripts.  This can be invoked manually with
> >> "make flake8" or "make mypy" in test/, and is also added to the static
> >> checkers test set.  
> > I never used a Python linter, so I'm not sure if it's as bad as Go or
> > Rust linters taking the whole poetry away, as it happened for instance
> > in my most recent experience with 'cargo fmt':
> >
> >    https://github.com/AsahiLinux/muvm/compare/68094c02c19b6f5d5e3def6d29379c1244c9a5e4..9af11c334a1ce37f533c056d982f8608c8d80d27#diff-e1a95ce380b9a8a317f97cccce1cbfd3dccd343dc62169ed1340208ab304fab9L106
> >
> >    https://github.com/AsahiLinux/muvm/pull/111#discussion_r1863551727
> >    (you need to click around before you get to it, no idea how to share
> >    a proper link that opens that comment right away)
>
> Not really on topic for this series but since you brought up rust there 
> is actually a "#[rustfmt::skip]" attribute that can be used to skip the 
> formatting selectively in case you need it in the future.

Ah, thanks, I didn't know about it. Now that you mention that, I just
found out that, with flake8, it looks like one can ignore the entire
file with "# flake8: noqa" on a line of its own:

  https://stackoverflow.com/a/64431741

but still, my doubts remain.

-- 
Stefano