update with ruff and manual code cleanup

.pre-commit-config.yaml

@@ -8,44 +8,33 @@ repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.6.0
     hooks:
-      - id: trailing-whitespace
       - id: check-added-large-files
         args: ["--maxkb=50000"]
       - id: check-ast
+      - id: check-case-conflict
       - id: check-json
       - id: check-merge-conflict
       - id: check-xml
       - id: check-yaml
       - id: debug-statements
       - id: end-of-file-fixer
+      - id: fix-byte-order-marker
       - id: requirements-txt-fixer
       - id: mixed-line-ending
         args: ["--fix=auto"]
-  - repo: https://github.com/pre-commit/mirrors-autopep8
-    rev: v2.0.4
-    hooks:
-      - id: autopep8
-        args:
-          [
-            "--in-place",
-            "--aggressive",
-            "--aggressive",
-            "--recursive",
-            "--max-line-length=100",
-            "--ignore=E501,E402,W503,W504,E731",
-          ]
-  - repo: https://github.com/pycqa/flake8
-    rev: 7.0.0
-    hooks:
-      - id: flake8
-        args: ["--ignore=E501,E402,W503,W504,E731,F401"]
+      # - id: pretty-format-json
+      #   args: ["--autofix", "--no-sort-keys", "--no-ensure-ascii"]
+      - id: trailing-whitespace
   - repo: https://github.com/pre-commit/mirrors-prettier
     rev: v4.0.0-alpha.8
     hooks:
       - id: prettier
-        types_or: [css, yaml, markdown, html, scss, javascript]
-  - repo: https://github.com/pre-commit/mirrors-isort
-    rev: v5.10.1
+        types_or: [css, yaml, markdown, html, scss, javascript, json]
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.4.1
     hooks:
-      - id: isort
-        args: ["-m=VERTICAL_HANGING_INDENT"] # vertical hanging
+      # Run the linter
+      - id: ruff
+        args: [--fix, --exit-non-zero-on-fix, --output-format=full]
+      # Run the formatter
+      - id: ruff-format

cspell.json

@@ -1,40 +1,40 @@
 {
-    "version": "0.2",
-    "ignorePaths": [],
-    "dictionaryDefinitions": [],
-    "dictionaries": [],
-    "words": [
-        "apibase",
-        "buildingsync",
-        "codeauthor",
-        "datafile",
-        "dname",
-        "durl",
-        "ECAM",
-        "EEEJ",
-        "ESPM",
-        "excpt",
-        "geocoded",
-        "greenbuildingregistry",
-        "jakejarvis",
-        "JSONAPI",
-        "jwalton",
-        "Munday",
-        "ndeloof",
-        "officedocument",
-        "openxmlformats",
-        "precommit",
-        "printenv",
-        "pyseed",
-        "pytest",
-        "sdist",
-        "SEEDO",
-        "seedrecords",
-        "spreadsheetml",
-        "subclassing",
-        "taxlot",
-        "taxlots"
-    ],
-    "ignoreWords": [],
-    "import": []
+  "version": "0.2",
+  "ignorePaths": [],
+  "dictionaryDefinitions": [],
+  "dictionaries": [],
+  "words": [
+    "apibase",
+    "buildingsync",
+    "codeauthor",
+    "datafile",
+    "dname",
+    "durl",
+    "ECAM",
+    "EEEJ",
+    "ESPM",
+    "excpt",
+    "geocoded",
+    "greenbuildingregistry",
+    "jakejarvis",
+    "JSONAPI",
+    "jwalton",
+    "Munday",
+    "ndeloof",
+    "officedocument",
+    "openxmlformats",
+    "precommit",
+    "printenv",
+    "pyseed",
+    "pytest",
+    "sdist",
+    "SEEDO",
+    "seedrecords",
+    "spreadsheetml",
+    "subclassing",
+    "taxlot",
+    "taxlots"
+  ],
+  "ignoreWords": [],
+  "import": []
 }

pyseed/__init__.py

@@ -1,5 +1,5 @@
 # Local Imports
-from pyseed.seed_client_base import (  # noqa
+from pyseed.seed_client_base import (  # noqa: F401
     SEEDOAuthReadOnlyClient,
     SEEDOAuthReadWriteClient,
     SEEDReadOnlyClient,