[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: |
Thu, 24 Sep 2020 15:29:04 -0400 |
On Wed, Sep 23, 2020 at 01:23:56PM -0400, John Snow wrote:
> On 9/23/20 9:42 AM, Cleber Rosa wrote:
> > 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>
> >
>
> Alright, I'll put it in the commit message itself instead of in the cover
> letter.
>
I missed that info on the cover letter, so my apologies. But still, I
think it's a good idea to have that preserved in the repo history
indeed.
> The next step is to re-engage on that Makefile patch that I was working on
> for ./python/qemu and introduce it here too, which will document the pinned
> versions correctly.
>
> --js
Agreed!
- Cleber.
signature.asc
Description: PGP signature
[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