[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 21/38] qapi/commands.py: enable checking with mypy
From: |
Cleber Rosa |
Subject: |
Re: [PATCH v2 21/38] qapi/commands.py: enable checking with mypy |
Date: |
Wed, 23 Sep 2020 18:21:18 -0400 |
On Tue, Sep 22, 2020 at 05:00:44PM -0400, John Snow wrote:
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> scripts/qapi/mypy.ini | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/scripts/qapi/mypy.ini b/scripts/qapi/mypy.ini
> index b668776d94..9da1dccef4 100644
> --- a/scripts/qapi/mypy.ini
> +++ b/scripts/qapi/mypy.ini
> @@ -4,11 +4,6 @@ strict_optional = False
> disallow_untyped_calls = False
> python_version = 3.6
>
> -[mypy-qapi.commands]
> -disallow_untyped_defs = False
> -disallow_incomplete_defs = False
> -check_untyped_defs = False
> -
> [mypy-qapi.doc]
> disallow_subclassing_any = False
> disallow_untyped_defs = False
> --
> 2.26.2
>
IMO, this increase of strictness for "commands" would make more sense
to be squashed together with the previous changes on "commands.py".
Not only here, but for the other patches for the other modules too.
Anyway,
Reviewed-by: Cleber Rosa <crosa@redhat.com>
signature.asc
Description: PGP signature
- Re: [PATCH v2 15/38] qapi/common.py: move build_params into gen.py, (continued)
[PATCH v2 26/38] qapi/gen.py: Enable checking with mypy, John Snow, 2020/09/22