Add actionlint to pre-commit

This allows us to stop manually linting shell scripts in our actions,
as actionlint will do this better (it understands the GitHub Actions
workflow syntax).

Author: whi-tw

.github/workflows/shell-ci.yml

@@ -22,29 +22,6 @@ jobs:
               -path "*/.terraform/*" \
           | xargs -L1 shellcheck
 
-      - name: Lint shell scripts in GitHub actions
-        run: |
-          # This is not the usual/best way to get the script directory.
-          # We need to do this here because GitHub Actions writes the
-          # content of script blocks like these to a temp directory,
-          # which means the source code directory and the directory
-          # in which the script reside are different
-          script_dir="$(readlink -f "$(dirname .github)")"
-
-          while IFS= read -r -d '' workflow
-          do
-            for path in $(yq '.jobs[] | .steps[] | select(.|has("run")) | .run | "."+(path | join("."))' "${script_dir}/${workflow}");
-            do
-              pushd "$(mktemp -d)" >/dev/null || exit
-                  workflow_name="$(basename "${script_dir}/${workflow}")"
-                  file="${workflow_name}__${path}"
-                  touch "${file}"
-                  yq "${path}" "${script_dir}/${workflow}" > "${file}"
-                  shellcheck -s bash "${file}"
-              popd >/dev/null || exit
-            done
-          done <  <(find ".github/workflows" -type f \( -name "*.yml" -or -name "*.yaml" \) -print0)
-
       - name: Lint shell scripts in CodeBuild build specs
         run: |
           ./support/shellcheck_codebuild.sh

.pre-commit-config.yaml

@@ -49,3 +49,8 @@ repos:
           - ruby
         language: system
         entry: make lint_ruby
+
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.7.7
+    hooks:
+      - id: actionlint