Merge pull request #52 from mbta/mth-shellcheck

ci: add shellcheck for shell scripts
mbta · Apr 26, 2024 · 648ecb8 · 648ecb8
2 parents 2f25aea + 83485c8
commit 648ecb8
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -13,9 +13,12 @@ jobs:
       - uses: actions/checkout@v4
       - run: yamllint -f parsable .
   chmod:
-    name: Ensure .sh files have chmod +x
+    name: Check shell scripts
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@v4
-      - run: find . -name '*.sh' \! -executable | (! grep .)
+      - name: shellcheck
+        run: find . -name '*.sh' | xargs shellcheck -S warning
+      - name: Ensure .sh files have chmod +x
+        run: find . -name '*.sh' \! -executable | (! grep .)