Merge pull request EOPF-Explorer#93 from d-v-b/chore/stronger-linting

chore/stronger linting

Author: d-v-b

.pre-commit-config.yaml

@@ -4,17 +4,11 @@ repos:
     hooks:
       - id: validate-pyproject
 
-  - repo: https://github.com/PyCQA/isort
-    rev: 6.0.1
-    hooks:
-      - id: isort
-        language_version: python
-
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.13.1
+    rev: v0.14.3
     hooks:
-      - id: ruff
-        args: ["--fix"]
+      - id: ruff-check
+        args: ["--fix", "--show-fixes"]
       - id: ruff-format
 
   - repo: https://github.com/pre-commit/mirrors-mypy

pyproject.toml

@@ -47,10 +47,6 @@ dependencies = [
 dev = [
     "pytest>=7.0.0",
     "pytest-cov>=4.0.0",
-    "black>=23.0.0",
-    "isort>=5.12.0",
-    "flake8>=6.0.0",
-    "flake8-docstrings>=1.7.0",
     "mypy>=1.0.0",
     "pre-commit>=3.0.0",
     "bandit[toml]>=1.7.0",
@@ -92,32 +88,62 @@ include = ["eopf_geozarr*"]
 [tool.setuptools.package-data]
 "*" = ["*.md", "*.txt", "*.yml", "*.yaml"]
 
-[tool.black]
+[tool.ruff]
 line-length = 100
-target-version = ['py310', 'py311', 'py312']
-include = '\.pyi?$'
-extend-exclude = '''
-/(
-  # directories
-  \.eggs
-  | \.git
-  | \.hg
-  | \.mypy_cache
-  | \.tox
-  | \.venv
-  | build
-  | dist
-)/
-'''
-
-[tool.isort]
-profile = "black"
-line_length = 100
-multi_line_output = 3
-include_trailing_comma = true
-force_grid_wrap = 0
-use_parentheses = true
-ensure_newline_before_comments = true
+force-exclude = true
+extend-exclude = [
+    ".bzr",
+    ".direnv",
+    ".eggs",
+    ".git",
+    ".mypy_cache",
+    ".nox",
+    ".pants.d",
+    ".ruff_cache",
+    ".venv",
+    "__pypackages__",
+    "_build",
+    "buck-out",
+    "build",
+    "dist",
+    "venv",
+    "docs",
+]
+
+[tool.ruff.lint]
+extend-select = [
+    "ANN",  # flake8-annotations
+    "B",    # flake8-bugbear
+    "C4",   # flake8-comprehensions
+    "EXE",  # flake8-executable
+    "FA",   # flake8-future-annotations
+    "FLY",  # flynt
+    "FURB", # refurb
+    "G",    # flake8-logging-format
+    "I",    # isort
+    "ISC",  # flake8-implicit-str-concat
+    "LOG",  # flake8-logging
+    "PERF", # Perflint
+    "PIE",  # flake8-pie
+    "PGH",  # pygrep-hooks
+    "PT",   # flake8-pytest-style
+    "PYI",  # flake8-pyi
+    "RET",  # flake8-return
+    "RSE",  # flake8-raise
+    "RUF",
+    "SIM",  # flake8-simplify
+    "SLOT", # flake8-slots
+    "TC",  # flake8-type-checking
+    "TRY",  # tryceratops
+    "UP",   # pyupgrade
+    "W",    # pycodestyle warnings
+]
+
+ignore = [
+    "ANN001",  # Missing type annotation for function argument - too strict for pytest fixtures
+    "ANN401",  # Dynamically typed Any - acceptable for **kwargs patterns
+    "TRY003",  # Long exception messages outside class - common pattern
+]
 
 [tool.mypy]
 python_version = "3.11"

src/eopf_geozarr/__init__.py

@@ -18,13 +18,13 @@
 
 __all__ = [
     "__version__",
-    "create_geozarr_dataset",
-    "setup_datatree_metadata_geozarr_spec_compliant",
-    "iterative_copy",
-    "consolidate_metadata",
     "async_consolidate_metadata",
-    "downsample_2d_array",
     "calculate_aligned_chunk_size",
+    "consolidate_metadata",
+    "create_geozarr_dataset",
+    "downsample_2d_array",
     "is_grid_mapping_variable",
+    "iterative_copy",
+    "setup_datatree_metadata_geozarr_spec_compliant",
     "validate_existing_band_data",
 ]