[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 07/38] qapi: add pylintrc
From: |
Cleber Rosa |
Subject: |
Re: [PATCH v2 07/38] qapi: add pylintrc |
Date: |
Wed, 23 Sep 2020 09:42:35 -0400 |
On Tue, Sep 22, 2020 at 05:00:30PM -0400, John Snow wrote:
> Using `pylint --generate-rcfile > pylintrc`, generate a skeleton
> pylintrc file. Sections that are not presently relevant (by the end of
> this series) are removed leaving just the empty section as a search
> engine / documentation hint to future authors.
>
> Right now, quite a few modules are ignored as they are known to fail as
> of this commit. modules 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 (for now). Due to a bug in pylint 2.5.x, pylint does not
> correctly recognize when it is being run from "inside" a package, and
> must be run *outside* of the package.
>
> Therefore, to run it, you must:
>
> > pylint scripts/qapi/ --rcfile=scripts/qapi/pylintrc
>
> Signed-off-by: John Snow <jsnow@redhat.com>
One concern I have here is that the pylint version is not defined.
Based on experience, different pylint will behave differently, because
among other things, it may introduce new checks.
I'd at the very least document the pylint version used in the commit
message, until a "requirements.txt"-like solution pinning a version is
given.
Other than that,
Reviewed-by: Cleber Rosa <crosa@redhat.com>
Tested-by: Cleber Rosa <crosa@redhat.com>
signature.asc
Description: PGP signature
- Re: [PATCH v2 02/38] qapi-gen: Separate arg-parsing from generation, (continued)
[PATCH v2 06/38] qapi: delint using flake8, John Snow, 2020/09/22
[PATCH v2 07/38] qapi: add pylintrc, John Snow, 2020/09/22
[PATCH v2 09/38] qapi/common.py: Add indent manager, John Snow, 2020/09/22
Re: [PATCH v2 09/38] qapi/common.py: Add indent manager, Cleber Rosa, 2020/09/23