Bump ruff to newest version and resulting changes (#1727)

The ruff version was quite old `0.8.0` vs `0.15.3`. I bumped the version
and needed a few small changes resulting from the new version.

Let me know if these changes are not desired and I can add `RUF059` to
the ignore list.

---------

Co-authored-by: Luiz Irber <luiz@earthmover.io>

Author: jbusecke

.pre-commit-config.yaml

@@ -10,7 +10,7 @@ repos:
       - id: mixed-line-ending
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: 'v0.8.0'
+    rev: 'v0.15.3'
     hooks:
       - id: ruff-format
       - id: ruff

icechunk-python/benchmarks/lib.py

@@ -60,7 +60,7 @@ def get_task_chunk_shape(
         if c == s or left is None:
             task_chunk_shape.append(c)
         else:
-            q, r = divmod(s, c)
+            q, _ = divmod(s, c)
             if q > left:
                 task_chunk_shape.append(left * c)
             else:

icechunk-python/pixi.lock

@@ -27,7 +27,7 @@ dependencies = ["zarr>=3.1.0"]
 dev = [
     { include-group = "test" },
     "mypy",
-    "ruff==0.8.0",
+    "ruff==0.15.3",
     "maturin>=1.7,<2.0",
     "maturin-import-hook>=0.3.0",
 ]

icechunk-python/pyproject.toml

@@ -314,6 +314,6 @@ def test_clear_virtual_chunk_containers_persists_through_reopen() -> None:
 
     # VCCs should be cleared after reopen
     reopened_vccs = repo.config.virtual_chunk_containers or {}
-    assert (
-        reopened_vccs == {}
-    ), f"Expected no VCCs after reopen, got: {list(reopened_vccs.keys())}"
+    assert reopened_vccs == {}, (
+        f"Expected no VCCs after reopen, got: {list(reopened_vccs.keys())}"
+    )

icechunk-python/tests/test_config.py

@@ -913,16 +913,16 @@ def expire_snapshots(
             )
 
         expired_but_remaining = actual & remaining_snapshot_ids
-        assert (
-            not expired_but_remaining
-        ), f"Snapshots marked as expired but still in ancestry: {expired_but_remaining}"
-
-        assert (
-            actual_deleted_branches == expected.deleted_branches
-        ), f"deleted branches mismatch: actual={actual_deleted_branches}, expected={expected.deleted_branches}"
-        assert (
-            actual_deleted_tags == expected.deleted_tags
-        ), f"deleted tags mismatch: actual={actual_deleted_tags}, expected={expected.deleted_tags}"
+        assert not expired_but_remaining, (
+            f"Snapshots marked as expired but still in ancestry: {expired_but_remaining}"
+        )
+
+        assert actual_deleted_branches == expected.deleted_branches, (
+            f"deleted branches mismatch: actual={actual_deleted_branches}, expected={expected.deleted_branches}"
+        )
+        assert actual_deleted_tags == expected.deleted_tags, (
+            f"deleted tags mismatch: actual={actual_deleted_tags}, expected={expected.deleted_tags}"
+        )
 
         for branch in actual_deleted_branches:
             self.maybe_checkout_branch(branch)