style: fix instances of A001

Signed-off-by: James McCorrie <james.mccorrie@lowrisc.org>

Author: machshev

ruff-ci.toml

@@ -23,7 +23,6 @@ ignore = [
 # High priority fixes
     "LOG015", # Using root logger
     "B008", # function calls in default arg value
-    "A001", # variable names mask builtins
     "PERF203", # Exception handling in loops
 
 # Temporary ignores just for CI until the rules pass then they can be removed

src/dvsim/launcher/sge/engine.py

@@ -31,9 +31,11 @@ def __init__(self, qstat_job_line) -> None:
         # Job line must have at least 8 tokens
         try:
             self.id = int(tokens[0])
+
         except ValueError:
             error_msg = "Could not convert data to an integer."
             raise ValueError(error_msg)
+
         self.priority = float(tokens[1])
         self.name = tokens[2]
         self.user = tokens[3]
@@ -78,12 +80,12 @@ def __init__(self, qstat_job_line) -> None:
         self._ja_tasklist = ja_task_id
 
     def __repr__(self) -> str:
-        repr = ["{"]
+        repr_str = ["{"]
         for key, value in self.__dict__.items():
             if key[0] != "_":
-                repr.append(key + "=" + str(value))
-        repr.append("}")
-        return "\n".join(repr)
+                repr_str.append(key + "=" + str(value))
+        repr_str.append("}")
+        return "\n".join(repr_str)
 
 
 class JobList:

src/dvsim/modes.py

@@ -1,6 +1,7 @@
 # Copyright lowRISC contributors (OpenTitan project).
 # Licensed under the Apache License, Version 2.0, see LICENSE for details.
 # SPDX-License-Identifier: Apache-2.0
+
 """Modes."""
 
 import logging as log