Merge pull request #1037 from ocefpaf/update_precommit_and_fix

Update precommit and fix

.pre-commit-config.yaml

@@ -25,23 +25,23 @@ repos:
     language_version: python3
 
 - repo: https://github.com/asottile/add-trailing-comma
-  rev: v2.4.0
+  rev: v3.0.0
   hooks:
     - id: add-trailing-comma
 
 
 - repo: https://github.com/charliermarsh/ruff-pre-commit
-  rev: v0.0.270
+  rev: v0.0.277
   hooks:
     - id: ruff
 
 - repo: https://github.com/tox-dev/pyproject-fmt
-  rev: 0.11.2
+  rev: 0.13.0
   hooks:
     - id: pyproject-fmt
 
 - repo: https://github.com/codespell-project/codespell
-  rev: v2.2.4
+  rev: v2.2.5
   hooks:
     - id: codespell
       args:

compliance_checker/cf/util.py

@@ -73,7 +73,6 @@
     "xlon",
     "XLON",
     "lonx",
-    "lonx",
     "lon_u",
     "LON_U",
     "lon_v",
@@ -97,7 +96,6 @@
     "ylat",
     "YLAT",
     "laty",
-    "laty",
     "lat_u",
     "LAT_U",
     "lat_v",

compliance_checker/tests/test_suite.py

@@ -146,7 +146,6 @@ def test_skip_check_level(self):
             "§3.5 flag_meanings for lat",
             "§3.5 flag_meanings for lon",
             "§3.5 lat is a valid flags variable",
-            "§3.5 lat is a valid flags variable",
             "§3.5 lon is a valid flags variable",
         }
 

pyproject.toml

@@ -6,12 +6,6 @@ requires = [
   "wheel",
 ]
 
-[tool.pytest.ini_options]
-markers = [
-    "integration: marks integration tests (deselect with '-m \"not integration\"')",
-    "slowtest: marks slow tests (deselect with '-m \"not slowtest\"')"
-]
-
 [tool.ruff]
 select = [
     "A", # flake8-builtins
@@ -40,3 +34,10 @@ ignore = [
   "A001",
 ]
 "compliance_checker/cfutil.py" = ["B028"]
+"compliance_checker/cf/appendix_f.py" = ["B033"]  # ignore duplicates items in the set
+
+[tool.pytest.ini_options]
+markers = [
+    "integration: marks integration tests (deselect with '-m \"not integration\"')",
+    "slowtest: marks slow tests (deselect with '-m \"not slowtest\"')"
+]