[PATCH v3 04/24] test/py: Fix some pylint warnings in test_ut.py

Tom Rini trini at konsulko.com
Fri Aug 16 18:15:08 CEST 2024


On Thu, Aug 15, 2024 at 02:25:15PM -0600, Simon Glass wrote:

> Tidy up most of these warnings. Remaining are four of these:
> 
>    R0914: Too many local variables
> 
> which can only by fixed by splitting things into functions, so that is
> left for another time.
> 
> Part of this change was done by the flynt tool.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
[snip]
> -    u_boot_utils.run_and_log(cons, 'qemu-img create %s 20M' % fname)
> -    u_boot_utils.run_and_log(cons, 'sudo sfdisk %s' % fname,
> +    u_boot_utils.run_and_log(cons, f'qemu-img create {fname} 20M')
> +    u_boot_utils.run_and_log(cons, f'sudo sfdisk {fname}',
>                               stdin=spec.encode('utf-8'))

This should be re-done on top of Richard's work (once he fixes the
pylint issues it introduces) at
https://patchwork.ozlabs.org/project/uboot/patch/20240802093322.15240-1-richard@nod.at/

-- 
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/20240816/94ae22d4/attachment.sig>


More information about the U-Boot mailing list