chore: update ruff for v0.8.0

.pre-commit-config.yaml

@@ -19,14 +19,8 @@ repos:
       - id: blacken-docs
         additional_dependencies: [black==24.*]
 
-  - repo: https://github.com/aio-libs/sort-all
-    rev: "v1.3.0"
-    hooks:
-      - id: sort-all
-        types: [file, python]
-
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.7.4"
+    rev: "v0.8.0"
     hooks:
       - id: ruff
         types: [file, python]

changelog/1225.contributor.rst

@@ -0,0 +1,4 @@
+Dropped `sort-all <https://github.com/aio-libs/sort-all>`__ ``pre-commit`` hook, as
+`ruff 0.8.0 <https://github.com/astral-sh/ruff/releases/tag/0.8.0>`__ stabilized the
+`unsorted-dunder-all <https://docs.astral.sh/ruff/rules/unsorted-dunder-all/>`__
+(``RUF022``) preview rule. (:user:`bjlittle`)

pyproject.toml

@@ -149,11 +149,6 @@ preview = false
 # Allow information-source (U+2139) which could be confused for "i".
 allowed-confusables = ["ℹ"]
 ignore = [
-  # flake8-annotations (ANN)
-  # https://docs.astral.sh/ruff/rules/#flake8-annotations-ann
-  "ANN101", # Missing type annotation for 'self' in method.
-  "ANN102", # Missing type annotation for 'cls' in classmethod.
-
   # flake8-commas (COM)
   # https://docs.astral.sh/ruff/rules/#flake8-commas-com
   "COM812", # Trailing comma missing.

src/geovista/bridge.py

@@ -54,8 +54,8 @@
     "BRIDGE_CLEAN",
     "NAME_CELLS",
     "NAME_POINTS",
-    "PathLike",
     "RIO_SIEVE_SIZE",
+    "PathLike",
     "Shape",
     "Transform",
 ]

src/geovista/common.py

@@ -46,9 +46,7 @@
     "GV_REMESH_POINT_IDS",
     "JUPYTER_BACKEND",
     "LRU_CACHE_SIZE",
-    "MixinStrEnum",
     "PERIOD",
-    "Preference",
     "RADIUS",
     "REMESH_JOIN",
     "REMESH_SEAM",
@@ -58,6 +56,8 @@
     "WRAP_RTOL",
     "ZLEVEL_SCALE",
     "ZTRANSFORM_FACTOR",
+    "MixinStrEnum",
+    "Preference",
     "active_kernel",
     "cast_UnstructuredGrid_to_PolyData",
     "distance",

src/geovista/core.py

@@ -56,8 +56,8 @@
 
 __all__ = [
     "CUT_OFFSET",
-    "MeridianSlice",
     "SPLINE_N_POINTS",
+    "MeridianSlice",
     "SliceBias",
     "add_texture_coords",
     "combine",

src/geovista/crs.py

@@ -27,9 +27,9 @@
 np = lazy.load("numpy")
 
 __all__ = [
+    "WGS84",
     "CRSLike",
     "PlateCarree",
-    "WGS84",
     "from_wkt",
     "get_central_meridian",
     "has_wkt",

src/geovista/geodesic.py

@@ -46,16 +46,16 @@
     "BBOX_C",
     "BBOX_RADIUS_RATIO",
     "BBOX_TOLERANCE",
-    "BBox",
-    "Corners",
     "ELLIPSE",
-    "EnclosedPreference",
     "GEODESIC_NPTS",
     "N_PANELS",
     "PANEL_BBOX_BY_IDX",
     "PANEL_IDX_BY_NAME",
     "PANEL_NAME_BY_IDX",
     "PREFERENCE",
+    "BBox",
+    "Corners",
+    "EnclosedPreference",
     "line",
     "npoints",
     "npoints_by_idx",

src/geovista/geoplotter.py

@@ -82,9 +82,9 @@
     "COASTLINES_RTOL",
     "GRATICULE_LABEL_FONT_SIZE",
     "GRATICULE_SHOW_LABELS",
+    "OPACITY_BLACKLIST",
     "GeoPlotter",
     "GeoPlotterBase",
-    "OPACITY_BLACKLIST",
 ]
 
 ADD_POINTS_STYLE: list[str, str] = ["points", "points_gaussian"]

src/geovista/gridlines.py

@@ -41,7 +41,6 @@
 __all__ = [
     "GRATICULE_CLOSED_INTERVAL",
     "GRATICULE_ZLEVEL",
-    "GraticuleGrid",
     "LABEL_DEGREE",
     "LABEL_EAST",
     "LABEL_NORTH",
@@ -59,6 +58,7 @@
     "LONGITUDE_STEP",
     "LONGITUDE_STEP_PERIOD",
     "LONGITUDE_STOP",
+    "GraticuleGrid",
     "create_meridian_labels",
     "create_meridians",
     "create_parallel_labels",

src/geovista/report.py

@@ -26,9 +26,9 @@
     "NCOL",
     "PACKAGES_CORE",
     "PACKAGES_OPTIONAL",
+    "TEXT_WIDTH",
     "PackageLike",
     "Report",
-    "TEXT_WIDTH",
 ]
 
 # type aliases

tests/core/test_slice_lines.py

@@ -177,6 +177,6 @@ def test_field_data(coastlines):
     metadata = dict(coastlines.field_data.items())
     result = slice_lines(coastlines)
     assert set(result.field_data.keys()) == set(metadata.keys())
-    for key in metadata:
-        assert id(result.field_data[key]) != id(metadata[key])
-        np.testing.assert_array_equal(result.field_data[key], metadata[key])
+    for key, value in metadata.items():
+        assert id(result.field_data[key]) != id(value)
+        np.testing.assert_array_equal(result.field_data[key], value)