chore: update pre-commit hooks (#4616)

* chore: update pre-commit hooks updates: - [github.com/astral-sh/ruff-pre-commit: v0.7.4 → v0.8.0](astral-sh/ruff-pre-commit@v0.7.4...v0.8.0) * style: pre-commit fixes --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
pybamm-team · Nov 25, 2024 · 403e1d3 · 403e1d3
1 parent dc5dbd3
commit 403e1d3
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 14 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,7 +4,7 @@ ci:
 
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.7.4"
+    rev: "v0.8.0"
     hooks:
       - id: ruff
         args: [--fix, --show-fixes]

diff --git a/tests/unit/test_solvers/test_idaklu_solver.py b/tests/unit/test_solvers/test_idaklu_solver.py
@@ -747,20 +747,24 @@ def test_setup_options(self):
                         options=options,
                     )
                     works = (
-                        jacobian == "none"
-                        and (linear_solver == "SUNLinSol_Dense")
-                        or jacobian == "dense"
-                        and (linear_solver == "SUNLinSol_Dense")
-                        or jacobian == "sparse"
-                        and (
-                            linear_solver != "SUNLinSol_Dense"
-                            and linear_solver != "garbage"
+                        (jacobian == "none" and (linear_solver == "SUNLinSol_Dense"))
+                        or (
+                            jacobian == "dense" and (linear_solver == "SUNLinSol_Dense")
                         )
-                        or jacobian == "matrix-free"
-                        and (
-                            linear_solver != "SUNLinSol_KLU"
-                            and linear_solver != "SUNLinSol_Dense"
-                            and linear_solver != "garbage"
+                        or (
+                            jacobian == "sparse"
+                            and (
+                                linear_solver != "SUNLinSol_Dense"
+                                and linear_solver != "garbage"
+                            )
+                        )
+                        or (
+                            jacobian == "matrix-free"
+                            and (
+                                linear_solver != "SUNLinSol_KLU"
+                                and linear_solver != "SUNLinSol_Dense"
+                                and linear_solver != "garbage"
+                            )
                         )
                     )