Merge pull request #238 from carlosmmatos/shfmt-workflow

chore: add workflow for shell formatting
CrowdStrike · Dec 21, 2023 · a49b5e5 · a49b5e5
2 parents 477dbb4 + 93abd73
commit a49b5e5
Show file tree

Hide file tree

Showing 4 changed files with 145 additions and 83 deletions.
diff --git a/.github/workflows/shell_format.yml b/.github/workflows/shell_format.yml
@@ -0,0 +1,24 @@
+on:
+  push:
+    paths:
+      - 'bash/**'
+      - '.github/workflows/shell_format.yml'
+
+  pull_request:
+    paths:
+      - 'bash/**'
+      - '.github/workflows/shell_format.yml'
+
+jobs:
+  shfmt:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Run shfmt with reviewdog
+      uses: reviewdog/[email protected]
+      with:
+        fail_on_error: true
+        workdir: 'bash'
+        shfmt_flags: '-i 4 -ci'
+
+
diff --git a/bash/containers/falcon-container-sensor-pull/falcon-container-sensor-pull.sh b/bash/containers/falcon-container-sensor-pull/falcon-container-sensor-pull.sh
@@ -306,7 +306,6 @@ case "${CONTAINER_TOOL}" in
     *) die "Unrecognized container runtime: ${CONTAINER_TOOL}" ;;
 esac
 
-
 # shellcheck disable=SC2005,SC2001
 cs_registry="registry.crowdstrike.com"
 if [ "${FALCON_CLOUD}" = "us-gov-1" ]; then