Ruff tweaks (#178)

* Use pyproject.toml for ruff config where possible

One less file to track, not strongly held opinion if
there are arguments against it

* Run ruff format before ruff check in precommit

That way any that can be fixed are checked before linting

.pre-commit-config.yaml

@@ -16,12 +16,10 @@ repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.1.6
     hooks:
+      - id: ruff-format
       - id: ruff
         args:
           - --fix
-      - id: ruff-format
-        args:
-          - --config=ruff.toml
 
   - repo: local
     hooks:

hasher/pyproject.toml

@@ -34,3 +34,10 @@ dev = [
 [build-system]
 requires = ["setuptools>=61.0"]
 build-backend = "setuptools.build_meta"
+
+[tool.ruff]
+extend = "../ruff.toml"
+
+[tool.ruff.extend-per-file-ignores]
+"./tests/**" = ["D1"]
+"./src/hasher/endpoints.py" = ["D103"]

pixl_core/pyproject.toml

@@ -38,3 +38,9 @@ build-backend = "setuptools.build_meta"
 
 [tool.pytest.ini_options]
 markers = ["pika"]
+
+[tool.ruff]
+extend = "../ruff.toml"
+
+[tool.ruff.extend-per-file-ignores]
+"./tests/**" = ["D100"]

pixl_ehr/pyproject.toml

@@ -39,5 +39,11 @@ build-backend = "setuptools.build_meta"
 [tool.pytest.ini_options]
 markers = ["processing"]
 
+[tool.ruff]
+extend = "../ruff.toml"
+
+[tool.ruff.extend-per-file-ignores]
+"./tests/**" = ["D1"]
+
 [tool.setuptools.package-data]
 pixl_ehr = ["sql/*.sql", "report_deid/*.txt"]

pixl_pacs/pyproject.toml

@@ -34,3 +34,9 @@ build-backend = "setuptools.build_meta"
 
 [tool.pytest.ini_options]
 markers = ["processing"]
+
+[tool.ruff]
+extend = "../ruff.toml"
+
+[tool.ruff.extend-per-file-ignores]
+"./tests/**" = ["D1"]