michaelh/zephyr
1
0
Fork 0
Code Releases Activity

code_relocation: fix ruff SIM102 warning

Browse source 
The ruff python linter produces a SIM102 warning when there are nested if
statements which can be collapsed into a single if statement:

    if foo:
        if bar:
	    ...

...becomes...

    if foo and bar:
        ...

This patch corrects the single instance of this issue in the script.

Signed-off-by: Joel Holdsworth <jholdsworth@nvidia.com>
This commit is contained in:
Joel Holdsworth 2025-05-21 10:17:15 -07:00 • committed by Benjamin Cabé
commit f4c54fcb93
2 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

1
.ruff-excludes.toml
View file

@ -234,7 +234,6 @@
]
"./scripts/build/gen_relocate_app.py" = [
"E101", # https://docs.astral.sh/ruff/rules/mixed-spaces-and-tabs
"SIM102", # https://docs.astral.sh/ruff/rules/collapsible-if
"SIM401", # https://docs.astral.sh/ruff/rules/if-else-block-instead-of-dict-get
"UP006", # https://docs.astral.sh/ruff/rules/non-pep585-annotation
"UP035", # https://docs.astral.sh/ruff/rules/deprecated-import

3
scripts/build/gen_relocate_app.py
View file

@ -551,8 +551,7 @@ def get_obj_filename(all_obj_files, filename):
obj_filename = filename.split("/")[-1] + ".obj"
for obj_file in all_obj_files:
if obj_file.name == obj_filename:
if filename.split("/")[-2] in obj_file.parent.name:
if obj_file.name == obj_filename and filename.split("/")[-2] in obj_file.parent.name:
return str(obj_file)