initial run of ruff

docs/conf.py

@@ -51,15 +51,14 @@
     "sphinx_copybutton",
     # For the card element
     "sphinx_design",
-    "sphinx_design", 
+    "sphinx_design",
     "sphinx.ext.autosummary",
     "sphinx.ext.mathjax",
     "sphinx.ext.githubpages",
     "matplotlib.sphinxext.plot_directive",
     "sphinx_copybutton",
     "IPython.sphinxext.ipython_directive",
     "IPython.sphinxext.ipython_console_highlighting",
-
     # So we can write markdown files
     "myst_parser",
 ]

docs/explanations/decisions/0002-switched-to-using-a-python-skeleton.rst

@@ -29,4 +29,4 @@ We will use the
 Consequences
 ------------
 
-CI workflows will change, even jobs providing the same feature as the older checks.
+CI workflows will change, even jobs providing the same feature as the older checks.

pyproject.toml

@@ -73,7 +73,7 @@ ignore_missing_imports = true # Ignore missing stubs in imported modules
 # Run pytest with all our checkers, and don't spam us with massive tracebacks on error
 addopts = """
     --tb=native -vv
-    --cov=event_model --cov-report term --cov-report xml:cov.xml
+    --cov=src/event_model --cov-report term --cov-report xml:cov.xml
     """
 # https://iscinumpy.gitlab.io/post/bound-version-constraints/#watch-for-warnings
 filterwarnings = "error"
@@ -106,8 +106,8 @@ allowlist_externals =
     sphinx-autobuild
 commands =
     pre-commit: pre-commit run --all-files --show-diff-on-failure {posargs}
-    type-checking: mypy event_model {posargs}
-    tests: pytest --cov=event_model --cov-report term --cov-report xml:cov.xml {posargs}
+    type-checking: mypy src/event_model {posargs}
+    tests: pytest --cov=src/event_model --cov-report term --cov-report xml:cov.xml {posargs}
     docs: sphinx-{posargs:build -E --keep-going} -T docs build/html
 """
 

src/event_model/__init__.py

@@ -2146,8 +2146,9 @@ def __call__(
     ) -> RunStop:
         if self.poison_pill:
             raise EventModelError(
-                "Already composed a RunStop document for run "
-                "{!r}.".format(self.start["uid"])
+                "Already composed a RunStop document for run " "{!r}.".format(
+                    self.start["uid"]
+                )
             )
         self.poison_pill.append(object())
         if uid is None:

src/event_model/documents/generate/__main__.py

@@ -3,11 +3,12 @@
 from event_model.documents import ALL_DOCUMENTS
 from event_model.documents.generate.typeddict_to_schema import typeddict_to_schema
 
+
 def main():
     schema_dir = Path(__file__).parent.parent.parent / "schemas"
     for document in ALL_DOCUMENTS:
         typeddict_to_schema(document, schema_dir)
 
 
 if __name__ == "__main__":
-   main() 
+    main()

src/event_model/schemas/datum.json

@@ -25,4 +25,4 @@
         "resource"
     ],
     "additionalProperties": false
-}
+}

src/event_model/schemas/datum_page.json

@@ -37,4 +37,4 @@
         "resource"
     ],
     "additionalProperties": false
-}
+}

src/event_model/schemas/event.json

@@ -58,4 +58,4 @@
         "uid"
     ],
     "additionalProperties": false
-}
+}

src/event_model/schemas/event_descriptor.json

@@ -278,4 +278,4 @@
         }
     },
     "additionalProperties": false
-}
+}

src/event_model/schemas/event_page.json

@@ -81,4 +81,4 @@
         "uid"
     ],
     "additionalProperties": false
-}
+}

src/event_model/schemas/resource.json

@@ -51,4 +51,4 @@
         "uid"
     ],
     "additionalProperties": false
-}
+}

src/event_model/schemas/run_start.json

@@ -357,4 +357,4 @@
         }
     },
     "additionalProperties": false
-}
+}

src/event_model/schemas/run_stop.json

@@ -63,4 +63,4 @@
         }
     },
     "additionalProperties": false
-}
+}

src/event_model/schemas/stream_datum.json

@@ -58,4 +58,4 @@
         "uid"
     ],
     "additionalProperties": false
-}
+}

src/event_model/schemas/stream_resource.json

@@ -42,4 +42,4 @@
         "uri"
     ],
     "additionalProperties": false
-}
+}