Re: [PATCH 2/8] python: update pylint ignores

[Markus Armbruster]

John Snow <jsnow@redhat.com> writes:

> The next patch will synchronize the qemu.qmp library with the external,
> standalone version. That synchronization will require a few extra
> ignores for pylint, so do that now.

Just one, unless I'm cross-eyed again.

> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
>  python/setup.cfg | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/python/setup.cfg b/python/setup.cfg
> index d7f5dc7bafe..02cc076eaea 100644
> --- a/python/setup.cfg
> +++ b/python/setup.cfg
> @@ -140,11 +140,12 @@ ignore_missing_imports = True
>  # --disable=W".
>  disable=consider-using-f-string,
>          consider-using-with,
> +        no-member,  # mypy also handles this better.
>          too-many-arguments,
> -        too-many-positional-arguments,
>          too-many-function-args,  # mypy handles this with less false 
> positives.
>          too-many-instance-attributes,
> -        no-member,  # mypy also handles this better.
> +        too-many-positional-arguments,

Up to here, just a reorder, which I appreciate.

> +        unknown-option-value,

This is the new one.

>  
>  [pylint.basic]
>  # Good variable names which should always be accepted, separated by a comma.