Add pre-commit checks and align lint/style configuration

snape · snape · commit 51705e54664d · 2026-04-04T19:20:59.000-04:00
Introduce a pre-commit setup to enforce repository quality checks and
normalize related formatting/lint behavior across the project.

- add pre-commit configuration with hooks for:
  - REUSE license linting
  - YAML linting
  - codespell
  - standard pre-commit hygiene checks
  - GitHub Actions workflow linting (actionlint)
- apply small consistency cleanups:
  - remove trailing whitespace/newline-only diffs in metadata/workflow files
  - normalize JSON formatting in metadata
diff --git a/.dockerignore b/.dockerignore
@@ -42,4 +42,3 @@ bazel-*
 build/
 cmake-build-*/
 CMakeUserPresets.json
-
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -96,7 +96,7 @@ jobs:
           wget -qO /usr/local/bin/bazelisk \
             https://github.com/bazelbuild/bazelisk/releases/latest/download/bazelisk-linux-amd64
           chmod +x /usr/local/bin/bazelisk
-          ln -s /usr/local/bin/bazelisk /usr/local/bin/bazel  
+          ln -s /usr/local/bin/bazelisk /usr/local/bin/bazel
         shell: bash
       - name: checkout
         uses: actions/checkout@v6
@@ -158,7 +158,7 @@ jobs:
   bazel-opensuse:
     name: bazel opensuse leap amd64
     runs-on: ubuntu-latest
-    container: opensuse/leap:15
+    container: opensuse/leap:16.0
     steps:
       - name: setup
         run: |
@@ -278,7 +278,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -313,7 +313,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -350,7 +350,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -389,7 +389,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -424,7 +424,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -438,7 +438,7 @@ jobs:
   cmake-opensuse:
     name: cmake opensuse leap amd64
     runs-on: ubuntu-latest
-    container: opensuse/leap:15
+    container: opensuse/leap:16.0
     steps:
       - name: setup
         run: |
@@ -462,7 +462,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -507,7 +507,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -552,7 +552,7 @@ jobs:
           cmake \
             -DBUILD_DOCUMENTATION:BOOL=ON \
             -DBUILD_TESTING:BOOL=ON \
-            -DCMAKE_INSTALL_PREFIX:PATH=${GITHUB_WORKSPACE}/_build/install \
+            -DCMAKE_INSTALL_PREFIX:PATH=install \
             -DWARNINGS_AS_ERRORS:BOOL=ON \
             --warn-uninitialized \
             ..
@@ -739,7 +739,7 @@ jobs:
   gradle-opensuse:
     name: gradle opensuse leap amd64
     runs-on: ubuntu-latest
-    container: opensuse/leap:15
+    container: opensuse/leap:16.0
     steps:
       - name: setup
         run: |
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -0,0 +1,43 @@
+---
+repos:
+  - repo: https://codeberg.org/fsfe/reuse-tool
+    rev: v6.2.0
+    hooks:
+      - id: reuse-lint-file
+
+  - repo: https://github.com/adrienverge/yamllint
+    rev: v1.38.0
+    hooks:
+      - id: yamllint
+        args:
+          - --format
+          - parsable
+
+  - repo: https://github.com/codespell-project/codespell
+    rev: v2.4.2
+    hooks:
+      - id: codespell
+
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      - id: check-case-conflict
+      - id: check-json
+      - id: check-merge-conflict
+      - id: check-yaml
+      - id: end-of-file-fixer
+      - id: trailing-whitespace
+        exclude: (?i)\.(markdown|md)$
+
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.7.12
+    hooks:
+      - id: actionlint
+        args:
+          - -oneline
+
+
+
+
+
+
diff --git a/.zenodo.json b/.zenodo.json
@@ -4,7 +4,7 @@
   "creators": [
     {
       "name": "Van den Berg, Jur",
-      "affiliation": "University of North Carolina at Chapel Hill",
+      "affiliation": "University of North Carolina at Chapel Hill"
     },
     {
       "name": "Guy, Stephen J.",
@@ -14,7 +14,7 @@
     {
       "name": "Snape, Jamie",
       "affiliation": "University of North Carolina at Chapel Hill",
-      "orcid": "0000-0002-3326-9765",
+      "orcid": "0000-0002-3326-9765"
     },
     {
       "name": "Lin, Ming C.",
diff --git a/blocks/.gitignore b/blocks/.gitignore
@@ -34,4 +34,5 @@
 # <https://gamma.cs.unc.edu/RVO2/>
 #
 
+/bin/
 /build/
diff --git a/circle/.gitignore b/circle/.gitignore
@@ -34,4 +34,5 @@
 # <https://gamma.cs.unc.edu/RVO2/>
 #
 
+/bin/
 /build/
diff --git a/rvo/.gitignore b/rvo/.gitignore
@@ -34,4 +34,5 @@
 # <https://gamma.cs.unc.edu/RVO2/>
 #
 
+/bin/
 /build/

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`"creators": [`
`5`	`5`	`{`
`6`	`6`	`"name": "Van den Berg, Jur",`
`7`		`- "affiliation": "University of North Carolina at Chapel Hill",`
	`7`	`+ "affiliation": "University of North Carolina at Chapel Hill"`
`8`	`8`	`},`
`9`	`9`	`{`
`10`	`10`	`"name": "Guy, Stephen J.",`
`@@ -14,7 +14,7 @@`
`14`	`14`	`{`
`15`	`15`	`"name": "Snape, Jamie",`
`16`	`16`	`"affiliation": "University of North Carolina at Chapel Hill",`
`17`		`- "orcid": "0000-0002-3326-9765",`
	`17`	`+ "orcid": "0000-0002-3326-9765"`
`18`	`18`	`},`
`19`	`19`	`{`
`20`	`20`	`"name": "Lin, Ming C.",`
Original file line number	Diff line number	Diff line change
`@@ -34,4 +34,5 @@`
`34`	`34`	`# <https://gamma.cs.unc.edu/RVO2/>`
`35`	`35`	`#`
`36`	`36`
	`37`	`+/bin/`
`37`	`38`	`/build/`