version 3

Author: Jakub Marciniak

.github/workflows/check-line-endings.yml

@@ -1,37 +1,42 @@
-name: Check Line Endings
+name: Check Shell Script Line Endings
 
 on:
-  pull_request:
-    paths:
-      - '**/*.sh'
   push:
-    paths:
-      - '**/*.sh'
+  pull_request:
 
 jobs:
-  check_line_endings:
+  check_crlf_in_sh:
+    name: Check for CRLF in .sh files
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v3
+        with:
+          unix-newlines: false
 
-      - name: Find shell scripts with CRLF line endings
+      - name: Scan all *.sh files for CRLF endings
         shell: bash
         run: |
-          echo "🔍 Checking for CRLF endings in shell scripts..."
+          echo "🔍 Scanning all *.sh files for CRLF endings..."
 
-          # Szukamy plików .sh zawierających CRLF
-          mapfile -t crlf_files < <(find . -type f -name "*.sh" -exec grep -Il $'\r' {} +)
+          mapfile -t all_sh_files < <(find . -type f -name "*.sh")
 
-          if (( ${#crlf_files[@]} > 0 )); then
-            echo ""
-            echo "🚫 The following .sh files contain CRLF (Windows-style) line endings:"
-            for file in "${crlf_files[@]}"; do
-              echo " • $file"
+          bad_files=()
+
+          for file in "${all_sh_files[@]}"; do
+            if grep -q $'\r' "$file"; then
+              bad_files+=("$file")
+            fi
+          done
+
+          if (( ${#bad_files[@]} > 0 )); then
+            echo "🚫 Found ${#bad_files[@]} shell script(s) with CRLF endings:"
+            for f in "${bad_files[@]}"; do
+              echo " • $f"
             done
             echo ""
-            echo "💡 Please convert them to LF endings using 'dos2unix' or 'shellman line_endings'."
+            echo "💡 Please convert these files to LF using 'dos2unix <file>' or 'shellman line_endings --file <file> --to lf'"
             exit 1
           else
-            echo "✅ All .sh files use correct LF line endings."
+            echo "✅ All .sh files use proper LF line endings."
           fi