Introduce github action linting in pre-commit

fedora-llvm-team · Feb 5, 2025 · 556b0d7 · 556b0d7
1 parent 82c989c
commit 556b0d7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.github/workflows/retest.yml b/.github/workflows/retest.yml
@@ -68,9 +68,11 @@ jobs:
     steps:
       - name: Get Chroots
         id: chroots-step
+        env:
+          COMMENT_BODY: ${{ github.event.comment.body }}
         run: |
-          echo "${{ github.event.comment.body }}" | grep -Pe '^\s*/retest\s+'
-          chroots=$(echo "${{ github.event.comment.body }}" | sed 's/^\s*\/retest\s*//g')
+          echo "$COMMENT_BODY" | grep -Pe '^\s*/retest\s+'
+          chroots=$(echo "$COMMENT_BODY" | sed 's/^\s*\/retest\s*//g')
           echo "chroots=$chroots" >> "$GITHUB_ENV"
       - uses: actions/checkout@v4
       - uses: ./.github/actions/prepare-python

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -55,4 +55,10 @@ repos:
         # https://pre-commit.com/#top_level-default_language_version
         language_version: python3.12
         force-exclude: "^snapshot_manager/tests/(test_logs|testing-farm-logs)/"
+
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.7.7
+    hooks:
+    - id: actionlint
+
 exclude: "^(snapshot_manager/tests/(test_logs|testing-farm-logs)/|media)"