Pre commit update

.pre-commit-config.yaml

@@ -1,7 +1,8 @@
 repos:
-  - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.11.0"
-    hooks:
-      - id: ruff
-        args: [--fix, --exit-non-zero-on-fix, --config=pyproject.toml]
-      - id: ruff-format
+- repo: https://github.com/astral-sh/ruff-pre-commit
+  rev: v0.12.7
+  hooks:
+    # Run the linter.
+    - id: ruff-check
+    # Run the formatter.
+    - id: ruff-format

codeflash/benchmarking/instrument_codeflash_trace.py

@@ -32,12 +32,14 @@ def leave_ClassDef(
     def visit_ClassDef(self, node: ClassDef) -> Optional[bool]:
         if self.class_name:  # Don't go into nested class
             return False
-        self.class_name = node.name.value  # noqa: RET503
+        self.class_name = node.name.value
+        return None
 
     def visit_FunctionDef(self, node: FunctionDef) -> Optional[bool]:
         if self.function_name:  # Don't go into nested function
             return False
-        self.function_name = node.name.value  # noqa: RET503
+        self.function_name = node.name.value
+        return None
 
     def leave_FunctionDef(self, original_node: FunctionDef, updated_node: FunctionDef) -> FunctionDef:
         if self.function_name == original_node.name.value:

codeflash/cli_cmds/cmd_init.py

@@ -38,7 +38,7 @@
     from argparse import Namespace
 
 CODEFLASH_LOGO: str = (
-    f"{LF}"  # noqa: ISC003
+    f"{LF}"
     r"                   _          ___  _               _     " + f"{LF}"
     r"                  | |        / __)| |             | |    " + f"{LF}"
     r"  ____   ___    _ | |  ____ | |__ | |  ____   ___ | | _  " + f"{LF}"

codeflash/models/models.py

@@ -480,7 +480,7 @@ def unique_invocation_loop_id(self) -> str:
         return f"{self.loop_index}:{self.id.id()}"
 
 
-class TestResults(BaseModel):
+class TestResults(BaseModel):  # noqa: PLW1641
     # don't modify these directly, use the add method
     # also we don't support deletion of test results elements - caution is advised
     test_results: list[FunctionTestInvocation] = []

codeflash/result/critic.py

@@ -1,6 +1,6 @@
 from __future__ import annotations
 
-from typing import TYPE_CHECKING, Optional
+from typing import TYPE_CHECKING
 
 from codeflash.cli_cmds.console import logger
 from codeflash.code_utils import env_utils
@@ -29,7 +29,8 @@ def speedup_critic(
     candidate_result: OptimizedCandidateResult,
     original_code_runtime: int,
     best_runtime_until_now: int | None,
-    disable_gh_action_noise: Optional[bool] = None,
+    *,
+    disable_gh_action_noise: bool = False,
 ) -> bool:
     """Take in a correct optimized Test Result and decide if the optimization should actually be surfaced to the user.
 

codeflash/verification/parse_test_output.py

@@ -271,7 +271,7 @@ def parse_test_xml(
                 groups = match.groups()
                 if len(groups[5].split(":")) > 1:
                     iteration_id = groups[5].split(":")[0]
-                    groups = groups[:5] + (iteration_id,)
+                    groups = (*groups[:5], iteration_id)
                 end_matches[groups] = match
 
             if not begin_matches or not begin_matches:

pyproject.toml

@@ -239,7 +239,8 @@ ignore = [
     "D104",
     "PERF203",
     "LOG015",
-    "PLC0415"
+    "PLC0415",
+    "UP045"
 ]
 
 [tool.ruff.lint.flake8-type-checking]