cicd: add precommit checks to GitHub actions (#621)

I noticed in #622 that the latest commit on this branch seemed to be in violation of some precommit hooks, so small changes to the schema source were introducing downstream changes to a bunch of unrelated def/json files.

This PR

* catches up on those changes and
* adds a precommit check to github actions to help contributors remember to run appropriate make commands/formatting hooks in their editors (or to install precommit themselves).

Author: jsstevenson

.github/workflows/cqa.yaml

@@ -0,0 +1,24 @@
+name: checks
+on: [push, pull_request]
+jobs:
+  precommit_hooks:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        cmd:
+          - "check-added-large-files"
+          - "trailing-whitespace"
+          - "end-of-file-fixer"
+          - "mixed-line-ending"
+          - "update-json-def-files"
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Set up Python 3.12
+        uses: actions/setup-python@v5
+        with:
+          python-version: 3.12
+
+      - uses: pre-commit/[email protected]
+        with:
+          extra_args: ${{ matrix.cmd }} --all-files

.pre-commit-config.yaml

@@ -6,6 +6,8 @@ repos:
       - id: detect-private-key
       - id: trailing-whitespace
       - id: end-of-file-fixer
+      - id: mixed-line-ending
+        args: [ --fix=lf ]
   - repo: local
     hooks:
       - id: update-json-def-files

.readthedocs.yaml

@@ -13,4 +13,4 @@ sphinx:
 
 python:
   install:
-    - requirements: docs/source/requirements.txt
+    - requirements: docs/source/requirements.txt

CONTRIBUTING.md

@@ -1,17 +1,17 @@
 # Contributing
 Contributions to this repository are intended to follow the VRS
 [development process](https://vrs.ga4gh.org/en/stable/appendices/development_process.html).
-The additional information presented here are guidelines for issues, 
-branches, commits, and pull requests. Before adding documentation, 
+The additional information presented here are guidelines for issues,
+branches, commits, and pull requests. Before adding documentation,
 please also review the [docs style guide](docs/source/style.rst).
 
 ## Discussions
-[Discussions](https://github.com/ga4gh/vrs/discussions) are for feature 
+[Discussions](https://github.com/ga4gh/vrs/discussions) are for feature
 requests, release candidate discussions, and questions.
 
 ## Issues
 [Issues](https://github.com/ga4gh/vrs/issues) are for bug
-reports, and planned feature descriptions. When creating an issue, use 
+reports, and planned feature descriptions. When creating an issue, use
 sentence case for the issue title and avoid the use of periods at the end
 of titles.
 
@@ -25,12 +25,12 @@ branch for [issue 250](https://github.com/ga4gh/vrs/issues/250) could
 be `250-contributing`.
 
 ## Pull Requests
-[Pull Requests](https://github.com/ga4gh/vrs/pulls) (PRs) for new 
-features should target the `main` branch. For version 
+[Pull Requests](https://github.com/ga4gh/vrs/pulls) (PRs) for new
+features should target the `main` branch. For version
 patches, the PR should target the appropriate minor version branch.
 PRs must be approved by at least one project maintainer before they may
 be merged. PR titles must reflect the issue associated with the PR. For
-example, the associated PR title for 
+example, the associated PR title for
 [issue 250](https://github.com/ga4gh/vrs/issues/250) would be
-`#250: Add CONTRIBUTING.md`, as seen in 
+`#250: Add CONTRIBUTING.md`, as seen in
 [PR #253](https://github.com/ga4gh/vrs/pull/253).

CONTRIBUTORS.md

@@ -42,7 +42,7 @@
 |Brian Walsh | [[10](#10)] |
 |Andrew D Yates | [[8](#8)] |
 
-See also 
+See also
 [VRS contributors](https://github.com/ga4gh/vrs/graphs/contributors) and
 [VRS Python contributors](https://github.com/ga4gh/vrs-python/graphs/contributors).
 

README.md

@@ -32,7 +32,7 @@ The VRS model is the product of the [GA4GH Variation Representation group](https
 
 ## Using the schema
 
-The schema is available in the [schema/](./schema/) directory, in both yaml and json versions. 
+The schema is available in the [schema/](./schema/) directory, in both yaml and json versions.
 It conforms to JSON Schema Draft 2020-12. For a list of
 libraries that support JSON schema, see
 [JSONSchema>Tools](https://json-schema.org/tools).

TODO

@@ -1,5 +1,5 @@
 # Docs
 see doc-updates branch
 * Standardize quoting: '**blah**' → ``blah``
-* Investigate 
-https://pypi.org/project/sphinx-jsonschema/
+* Investigate
+https://pypi.org/project/sphinx-jsonschema/

docs/source/appendices/design_decisions.rst

@@ -9,14 +9,14 @@ GA4GH Inherent Properties over Value Objects
 --------------------------------------------
 
 In VRS 1.0 we operated under the principle that all identifiable objects in VRS (e.g. Allele, SequenceLocation, etc.)
-would be *value objects*. This meant that they should be immutable and contain only required fields that are 
+would be *value objects*. This meant that they should be immutable and contain only required fields that are
 necessary to uniquely identify the object. This approach somewhat simplified the ability to genertate the digests by
 allowing the computation of the digest to be based on the entire object. An exception was made for properties with a
 leading underscore (namely, the *_id* property), which was removed from the object before a digest was calculated.
 
 In VRS 2.0 we extended the principle of excepting designated attributes by explicitly defining *inherent properties*
-that constitute the properties used to compute an object digest. This was done to enable expressivity of VRS, 
-enabling implementations to pass common, descriptive metadata as part of the identifiable objects without sacrificing 
+that constitute the properties used to compute an object digest. This was done to enable expressivity of VRS,
+enabling implementations to pass common, descriptive metadata as part of the identifiable objects without sacrificing
 the ability to create globally unique, federated identifiers from VRS 1.3.
 
 As a result, we had to introduce a new field in the digest model called *ga4gh.inherent* which is described in detail
@@ -25,13 +25,13 @@ in the section on :ref:`ga4gh-inherent-properties`.
 IRIs over CURIEs
 ----------------
 
-In VRS 2.0 we moved away from the use of CURIEs in favor of :ref:`iriReference`. Several factors played a role in 
+In VRS 2.0 we moved away from the use of CURIEs in favor of :ref:`iriReference`. Several factors played a role in
 this decision.
 
-JSON Schema, the default data model for GKS specifications, does not allow for encoding of CURIE namespaces as is done 
-in other frameworks such as JSON-LD or XML. As a result, namespaces must be captured from custom data structures, API 
+JSON Schema, the default data model for GKS specifications, does not allow for encoding of CURIE namespaces as is done
+in other frameworks such as JSON-LD or XML. As a result, namespaces must be captured from custom data structures, API
 endpoints, or documentation that may not persist as messages are exchanged between systems. To address this, references
-in GKS specs now use IRIs to reference objects explicitly. 
+in GKS specs now use IRIs to reference objects explicitly.
 
 IRI-References over IRIs
 ------------------------
@@ -44,7 +44,7 @@ VRS identifier syntax and versioning
 
 The :ref:`versioning` section describes the versioning and release naming conventions for the VRS product.
 Approved releases will be assigned to the version number alone, but connect, ballot and snapshot releases will
-include the context term and date in addition to the target version number. 
+include the context term and date in addition to the target version number.
 
 During the GA4GH Connect April 2023 meeting the maturity model was discussed at length and the following
 proposal was presented for instance and class GKS identifiers.
@@ -64,13 +64,13 @@ As an example, the Github JSON Schema URL ($id) for the VRS 2.0.0 Allele is:
   }
 
 During the **release and versioning** discussion at the GA4GH Connect April 2023 meeting the proposal
-delved into the idea of including the major version number in the VRS identifier itself. Proponents of 
-this approach cited concern for the change in digests (and their derived identifiers) between major 
-versions of the same VRS object, which would become clearly visible in the identifier itself if the 
+delved into the idea of including the major version number in the VRS identifier itself. Proponents of
+this approach cited concern for the change in digests (and their derived identifiers) between major
+versions of the same VRS object, which would become clearly visible in the identifier itself if the
 major version was included.
 
-Opponents of this approach argued that new identifiers would be required for every type of VRS object 
-for every major version release. Meaning that even if a given type of object has no change that would 
+Opponents of this approach argued that new identifiers would be required for every type of VRS object
+for every major version release. Meaning that even if a given type of object has no change that would
 result in a new digest, a new identifier would still be required for the new major version.
 
 After much discussion, the decision was made to NOT include the major version number in the VRS identifier
@@ -88,4 +88,3 @@ the following syntax:
 .. code-block::
 
   https://w3id.org/ga4gh/vrs/VA.Oop4kjdTtKcg1kiZjIJAAR3bp7qi4aNT
-

docs/source/appendices/ga4gh_identifiers.rst

@@ -79,8 +79,8 @@ GA4GH Inherent Properties
     implementations. VRS 2.0 addresses this limitation with the designation of inherent properties
     for use with the computed identifier algorithm.
 
-When creating computed identifiers from objects, VRS uses a custom schema attribute, 
-*ga4gh.inherent*, that contains the property names used for computing digests. For example, 
+When creating computed identifiers from objects, VRS uses a custom schema attribute,
+*ga4gh.inherent*, that contains the property names used for computing digests. For example,
 the Allele JSON Schema:
 
 .. parsed-literal::
@@ -105,7 +105,7 @@ the Allele JSON Schema:
 
 .. note::
 
-  The `ga4gh` JSON Schema namespace is aligned with the Sequence Collections effort 
+  The `ga4gh` JSON Schema namespace is aligned with the Sequence Collections effort
   (see `SeqCol#84 <https://github.com/ga4gh/refget/issues/84>`_).
 
 GA4GH Type Prefixes