Add true and split into array

Signed-off-by: Ryan Liang <jiallian@amazon.com>
RyanL1997 · Dec 21, 2023 · e726b9a · e726b9a
1 parent 165b0fc
commit e726b9a
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -63,15 +63,16 @@ jobs:
           HEAD_SHA="${{ github.event.pull_request.head.sha }}"
           git fetch origin $BASE_SHA
           git diff --name-only $BASE_SHA...$HEAD_SHA > changed_files.txt
-          CHANGED_FILES=$(cat changed_files.txt | grep -E '\.(js|ts|tsx)$')
+          CHANGED_FILES=$(cat changed_files.txt | grep -E '\.(js|ts|tsx)$' || true)
           echo "::set-output name=changed::${CHANGED_FILES}"
         
       - name: Lint Changed Files
         run: |
           CHANGED_FILES="${{ steps.files.outputs.changed }}"
-          if [ -n "$CHANGED_FILES" ]; then
+          if [[ -n "$CHANGED_FILES" && "$CHANGED_FILES" != "" ]]; then
             echo "Linting changed files..."
-            yarn lint $CHANGED_FILES
+            IFS=$'\n' read -r -a FILES_TO_LINT <<< "$CHANGED_FILES"
+            yarn lint "${FILES_TO_LINT[@]}"
           else
             echo "No JavaScript/TypeScript files to lint."
           fi