Re: [bitbake-devel][PATCH 1/1] utils: pylint: clean up imports

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Mon, 2025-02-17 at 16:29 +0100, Michael Estner via
lists.openembedded.org wrote:
> As mentioned in the TODO file I run pylint
> and start clean code including:
> Remove double imports
> Order the imports as mentioned by pylint.
> First small patch to see if what I do
> fits the expectations.
> 
> Signed-off-by: Michael Estner <michaelestner@web.de>
> ---
>  lib/bb/utils.py | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)

Thanks for this. The patch itself was fine and I've merged it. I did
need to tweak the commit message itself a little bit to be more focused
on the patch and have correct line wrapping.

I was initially a bit worried as the TODO file hasn't been touched in a
long time but you did pick a sensible improvement so well done for
filtering it to something useful.

I would warn that we wouldn't necessarily act on everything suggested
by the linter, we'd prefer to focus on "real" issues. Unneeded imports
definitely fit into that category.

I'll try and find some time to clean up that TODO file a bit.

Cheers,

Richard