[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 07/37] qapi: add pylintrc
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 07/37] qapi: add pylintrc |
Date: |
Wed, 16 Sep 2020 14:30:03 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
John Snow <jsnow@redhat.com> writes:
> Add a skeleton pylintrc file. Right now, it ignores quite a few things.
> Files will be removed from the known-bad list throughout this and
> following series as they are repaired.
>
> Note: Normally, pylintrc would go in the folder above the module, but as
> that folder is shared by many things, it is going inside the module
> folder now.
>
> Due to some bugs in different versions of pylint (2.5.x), pylint does
> not correctly recognize when it is being run from "inside" a module, and
> must be run *outside* of the module.
>
> Therefore, to run it, you must:
>
> > cd :/qemu/scripts
-bash: cd: :/qemu/scripts: No such file or directory
;-P
> > pylint qapi/ --rcfile=qapi/pylintrc
Why not
$ pylint scripts/qapi --rcfile=scripts/qapi/pylintrc
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> scripts/qapi/pylintrc | 74 +++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 74 insertions(+)
> create mode 100644 scripts/qapi/pylintrc
>
> diff --git a/scripts/qapi/pylintrc b/scripts/qapi/pylintrc
> new file mode 100644
> index 0000000000..c2bbb8e8e1
> --- /dev/null
> +++ b/scripts/qapi/pylintrc
> @@ -0,0 +1,74 @@
> +[MASTER]
> +
> +# Add files or directories matching the regex patterns to the blacklist. The
> +# regex matches against base names, not paths.
> +ignore-patterns=common.py,
> + doc.py,
> + error.py,
> + expr.py,
> + gen.py,
> + parser.py,
> + schema.py,
> + source.py,
> + types.py,
> + visit.py,
Already not ignored:
__init__.py
commands.py
common.py
debug.py
events.py
introspect.py
script.py
Okay.
> +
> +
> +[MESSAGES CONTROL]
> +
> +# Disable the message, report, category or checker with the given id(s). You
> +# can either give multiple identifiers separated by comma (,) or put this
> +# option multiple times (only on the command line, not in the configuration
> +# file where it should appear only once). You can also use "--disable=all" to
> +# disable everything first and then reenable specific checks. For example, if
> +# you want to run only the similarities checker, you can use "--disable=all
> +# --enable=similarities". If you want to run only the classes checker, but
> have
> +# no Warning level messages displayed, use "--disable=all --enable=classes
> +# --disable=W".
> +disable=fixme,
> + missing-docstring,
> + too-many-arguments,
> + too-many-branches,
> + too-many-statements,
> + too-many-instance-attributes,
I'm fine with disabling these.
> +
> +[REPORTS]
> +
> +[REFACTORING]
> +
> +[MISCELLANEOUS]
> +
> +[LOGGING]
> +
> +[BASIC]
> +
> +# Good variable names which should always be accepted, separated by a comma.
> +good-names=i,
> + j,
> + k,
> + ex,
> + Run,
> + _
Isn't this the default?
> +
> +[VARIABLES]
> +
> +[STRING]
> +
> +[SPELLING]
> +
> +[FORMAT]
> +
> +[SIMILARITIES]
> +
> +# Ignore imports when computing similarities.
> +ignore-imports=yes
Why?
> +
> +[TYPECHECK]
> +
> +[CLASSES]
> +
> +[IMPORTS]
> +
> +[DESIGN]
> +
> +[EXCEPTIONS]
Looks like you started with output of --generate-rcfile,
- Re: [PATCH 04/37] qapi: move generator entrypoint into module, (continued)
[PATCH 05/37] qapi: Remove wildcard includes, John Snow, 2020/09/15
[PATCH 06/37] qapi: delint using flake8, John Snow, 2020/09/15
[PATCH 07/37] qapi: add pylintrc, John Snow, 2020/09/15
- Re: [PATCH 07/37] qapi: add pylintrc,
Markus Armbruster <=
[PATCH 09/37] qapi/common.py: Add indent manager, John Snow, 2020/09/15
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, Markus Armbruster, 2020/09/16
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, John Snow, 2020/09/16
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, Markus Armbruster, 2020/09/17
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, John Snow, 2020/09/17
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, Markus Armbruster, 2020/09/18
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, John Snow, 2020/09/18