From cd02de5df90c8dbf6b4306948ca2971af9ff9c8e Mon Sep 17 00:00:00 2001 From: Pieter De Gendt Date: Wed, 20 Nov 2024 13:13:44 +0100 Subject: [PATCH] scripts: west_commands: runners: Fix reimplemented-builtin (SIM110) Convert for loop to use any(). See https://docs.astral.sh/ruff/rules/reimplemented-builtin/ Signed-off-by: Pieter De Gendt --- scripts/west_commands/runners/bossac.py | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/scripts/west_commands/runners/bossac.py b/scripts/west_commands/runners/bossac.py index 4526e95dd33..143246bb06e 100644 --- a/scripts/west_commands/runners/bossac.py +++ b/scripts/west_commands/runners/bossac.py @@ -76,19 +76,13 @@ class BossacBinaryRunner(ZephyrBinaryRunner): def supports(self, flag): """Check if bossac supports a flag by searching the help""" - for line in self.read_help(): - if flag in line: - return True - return False + return any(flag in line for line in self.read_help()) def is_extended_samba_protocol(self): ext_samba_versions = ['CONFIG_BOOTLOADER_BOSSA_ARDUINO', 'CONFIG_BOOTLOADER_BOSSA_ADAFRUIT_UF2'] - for x in ext_samba_versions: - if self.build_conf.getboolean(x): - return True - return False + return any(self.build_conf.getboolean(x) for x in ext_samba_versions) def is_partition_enabled(self): return self.build_conf.getboolean('CONFIG_USE_DT_CODE_PARTITION')