[PATCH 6/6] Azure/GitLab CI: Add the pylint checker

Simon Glass sjg at chromium.org
Thu Nov 25 01:13:12 CET 2021


Hi Heinrich,

On Mon, 22 Nov 2021 at 00:40, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> On 11/22/21 04:48, Simon Glass wrote:
> > Add a check that new Python code does not regress the pylint score for
> > any module.
>
> How will ./scripts/pylint.base be updated?

For the moment people will have to add it to their patches.

There is no automatic 'ratchet up' at present, though, if that is what
you are asking.

>
> Some or the worst scores are for binman. Any plans to address these?

Yes. A big one is changing the function naming to snake case. Another
is that classes override their parent methods and the comments are not
copied from the parent. I'm not sure whether we want to do that or
not.

Regards,
Simon


More information about the U-Boot mailing list