Merge pull request #105 from ssciwr/github-actions-linting

Add actionlint to pre-commit - a linter for GitHub Action workflow files
ssciwr · Feb 8, 2024 · 1c24c50 · 1c24c50
2 parents 5298564 + 44f63c4
commit 1c24c50
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 9 deletions.
diff --git a/{{cookiecutter.project_slug}}/.github/workflows/ci.yml b/{{cookiecutter.project_slug}}/.github/workflows/ci.yml
@@ -79,21 +79,21 @@ jobs:
 {%- endif %}
 
     - name: make build directory
-      run: cmake -E make_directory ${{ "{{runner.workspace}}" }}/build
+      run: cmake -E make_directory ${{ "{{ github.workspace }}" }}/build
 
     - name: configure cmake
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}/build
+      working-directory: ${{ "{{ github.workspace }}" }}/build
       run: cmake $GITHUB_WORKSPACE -DCMAKE_BUILD_TYPE=Debug -DBUILD_DOCS=OFF
 
     - name: build
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}/build
+      working-directory: ${{ "{{ github.workspace }}" }}/build
       run: cmake --build .
 
     - name: run tests
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}/build
+      working-directory: ${{ "{{ github.workspace }}" }}/build
       run: ctest
 {%- if cookiecutter.python_bindings == "Yes" %}
   python-testing:
@@ -183,26 +183,26 @@ jobs:
 {%- endif %}
 
     - name: make build directory
-      run: cmake -E make_directory ${{ "{{runner.workspace}}" }}/build
+      run: cmake -E make_directory ${{ "{{ github.workspace }}" }}/build
 
     - name: configure cmake
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}/build
+      working-directory: ${{ "{{ github.workspace }}" }}/build
       run: cmake $GITHUB_WORKSPACE -DCMAKE_CXX_FLAGS="--coverage" -DBUILD_DOCS=OFF
 
     - name: build
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}/build
+      working-directory: ${{ "{{ github.workspace }}" }}/build
       run: cmake --build .
 
     - name: run tests
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}/build
+      working-directory: ${{ "{{ github.workspace }}" }}/build
       run: ctest
 
     - name: collect coverage report
       shell: bash
-      working-directory: ${{ "{{runner.workspace}}" }}
+      working-directory: ${{ "{{ github.workspace }}" }}
       run: |
         lcov --directory ./build{% if cookiecutter.header_only == "No" %}/src{% endif %} --capture --output-file coverage.info
 

diff --git a/{{cookiecutter.project_slug}}/.pre-commit-config.yaml b/{{cookiecutter.project_slug}}/.pre-commit-config.yaml
@@ -39,3 +39,11 @@ repos:
     hooks:
       - id: cmake-format
       - id: cmake-lint
+{%- if cookiecutter.github_actions_ci == "Yes" %}
+
+  # GitHub Actions Workflow linter
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.6.25
+    hooks:
+      - id: actionlint
+{%- endif %}