michaelh/zephyr
1
0
Fork 0
Code Releases Activity

scripts: west_commands: zspdx: cmakefileapi: Fix linter issues

Browse source 
Fix issues reported by ruff.

Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
This commit is contained in:
Pieter De Gendt 2025-05-24 21:49:04 +02:00 • committed by Benjamin Cabé
commit 2913bfee6f
2 changed files with 18 additions and 22 deletions
Download patch file Download diff file Expand all files Collapse all files

5
.ruff-excludes.toml
View file

@ -1190,11 +1190,6 @@
"UP015", # https://docs.astral.sh/ruff/rules/redundant-open-modes "UP015", # https://docs.astral.sh/ruff/rules/redundant-open-modes
"UP032", # https://docs.astral.sh/ruff/rules/f-string "UP032", # https://docs.astral.sh/ruff/rules/f-string
] ]
"./scripts/west_commands/zspdx/cmakefileapi.py" = [
"F541", # https://docs.astral.sh/ruff/rules/f-string-missing-placeholders
"I001", # https://docs.astral.sh/ruff/rules/unsorted-imports
"UP008", # https://docs.astral.sh/ruff/rules/super-call-with-parameters
]
"./scripts/west_commands/zspdx/cmakefileapijson.py" = [ "./scripts/west_commands/zspdx/cmakefileapijson.py" = [
"E501", # https://docs.astral.sh/ruff/rules/line-too-long "E501", # https://docs.astral.sh/ruff/rules/line-too-long
"F541", # https://docs.astral.sh/ruff/rules/f-string-missing-placeholders "F541", # https://docs.astral.sh/ruff/rules/f-string-missing-placeholders

35
scripts/west_commands/zspdx/cmakefileapi.py
View file

@ -4,10 +4,11 @@
from enum import Enum from enum import Enum
class Codemodel: class Codemodel:
def __init__(self): def __init__(self):
super(Codemodel, self).__init__() super().__init__()
self.paths_source = "" self.paths_source = ""
self.paths_build = "" self.paths_build = ""
@ -20,7 +21,7 @@ class Codemodel:
class Config: class Config:
def __init__(self): def __init__(self):
super(Config, self).__init__() super().__init__()
self.name = "" self.name = ""
self.directories = [] self.directories = []
@ -29,7 +30,7 @@ class Config:
def __repr__(self): def __repr__(self):
if self.name == "": if self.name == "":
return f"Config: [no name]" return "Config: [no name]"
else: else:
return f"Config: {self.name}" return f"Config: {self.name}"
@ -37,7 +38,7 @@ class Config:
class ConfigDir: class ConfigDir:
def __init__(self): def __init__(self):
super(ConfigDir, self).__init__() super().__init__()
self.source = "" self.source = ""
self.build = "" self.build = ""
@ -61,7 +62,7 @@ class ConfigDir:
class ConfigProject: class ConfigProject:
def __init__(self): def __init__(self):
super(ConfigProject, self).__init__() super().__init__()
self.name = "" self.name = ""
self.parentIndex = -1 self.parentIndex = -1
@ -82,7 +83,7 @@ class ConfigProject:
class ConfigTarget: class ConfigTarget:
def __init__(self): def __init__(self):
super(ConfigTarget, self).__init__() super().__init__()
self.name = "" self.name = ""
self.id = "" self.id = ""
@ -114,7 +115,7 @@ class TargetType(Enum):
class TargetInstallDestination: class TargetInstallDestination:
def __init__(self): def __init__(self):
super(TargetInstallDestination, self).__init__() super().__init__()
self.path = "" self.path = ""
self.backtrace = -1 self.backtrace = -1
@ -127,7 +128,7 @@ class TargetInstallDestination:
class TargetCommandFragment: class TargetCommandFragment:
def __init__(self): def __init__(self):
super(TargetCommandFragment, self).__init__() super().__init__()
self.fragment = "" self.fragment = ""
self.role = "" self.role = ""
@ -139,7 +140,7 @@ class TargetCommandFragment:
class TargetDependency: class TargetDependency:
def __init__(self): def __init__(self):
super(TargetDependency, self).__init__() super().__init__()
self.id = "" self.id = ""
self.backtrace = -1 self.backtrace = -1
@ -151,7 +152,7 @@ class TargetDependency:
class TargetSource: class TargetSource:
def __init__(self): def __init__(self):
super(TargetSource, self).__init__() super().__init__()
self.path = "" self.path = ""
self.compileGroupIndex = -1 self.compileGroupIndex = -1
@ -170,7 +171,7 @@ class TargetSource:
class TargetSourceGroup: class TargetSourceGroup:
def __init__(self): def __init__(self):
super(TargetSourceGroup, self).__init__() super().__init__()
self.name = "" self.name = ""
self.sourceIndexes = [] self.sourceIndexes = []
@ -185,7 +186,7 @@ class TargetSourceGroup:
class TargetCompileGroupInclude: class TargetCompileGroupInclude:
def __init__(self): def __init__(self):
super(TargetCompileGroupInclude, self).__init__() super().__init__()
self.path = "" self.path = ""
self.isSystem = False self.isSystem = False
@ -198,7 +199,7 @@ class TargetCompileGroupInclude:
class TargetCompileGroupPrecompileHeader: class TargetCompileGroupPrecompileHeader:
def __init__(self): def __init__(self):
super(TargetCompileGroupPrecompileHeader, self).__init__() super().__init__()
self.header = "" self.header = ""
self.backtrace = -1 self.backtrace = -1
@ -210,7 +211,7 @@ class TargetCompileGroupPrecompileHeader:
class TargetCompileGroupDefine: class TargetCompileGroupDefine:
def __init__(self): def __init__(self):
super(TargetCompileGroupDefine, self).__init__() super().__init__()
self.define = "" self.define = ""
self.backtrace = -1 self.backtrace = -1
@ -222,7 +223,7 @@ class TargetCompileGroupDefine:
class TargetCompileGroup: class TargetCompileGroup:
def __init__(self): def __init__(self):
super(TargetCompileGroup, self).__init__() super().__init__()
self.sourceIndexes = [] self.sourceIndexes = []
self.language = "" self.language = ""
@ -242,7 +243,7 @@ class TargetCompileGroup:
class TargetBacktraceGraphNode: class TargetBacktraceGraphNode:
def __init__(self): def __init__(self):
super(TargetBacktraceGraphNode, self).__init__() super().__init__()
self.file = -1 self.file = -1
self.line = -1 self.line = -1
@ -257,7 +258,7 @@ class TargetBacktraceGraphNode:
class Target: class Target:
def __init__(self): def __init__(self):
super(Target, self).__init__() super().__init__()
self.name = "" self.name = ""
self.id = "" self.id = ""