[PATCH 2/2] u_boot_pylib: Clean up pylint warnings in gitutil.py

[Tom Rini]

On Fri, Mar 14, 2025 at 02:45:27PM +0000, Simon Glass wrote:
> Hi Tom,
> 
> On Mon, 17 Feb 2025 at 19:23, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Sun, Feb 09, 2025 at 02:26:01PM -0700, Simon Glass wrote:
> >
> > > Reduce the number of warnings in this file.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > ---
> > >
> > >  tools/buildman/func_test.py   |   4 +-
> > >  tools/u_boot_pylib/gitutil.py | 240 ++++++++++++++++++----------------
> > >  2 files changed, 128 insertions(+), 116 deletions(-)
> >
> > As I think these all were warnings from whatever other linting you had
> > mentioned in another thread, I deferred this patch as it also doesn't
> > cleanly apply.
> 
> Some of it, yes. But most of it is fixing things that we should fix.

To be clear, to the extent any of it was pylint problems, they've now
been addressed as we've moved to pylint 3.3.4.

> This is going to open up a significant diff for patman, unfortunately,
> just as I am adding new features for series management. So please
> consider applying it. I can rebase if you like?

Yes, that's fine, please just explain what's showing the warnings or
otherwise update the commit message.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250314/de0bf561/attachment.sig>