ci: stricter filename checking in diff generation (#3801)

GaloyMoney · Jan 9, 2024 · 4232a5b · 4232a5b
1 parent 0843e94
commit 4232a5b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/ci/apps/tasks/open-charts-pr.sh b/ci/apps/tasks/open-charts-pr.sh
@@ -40,7 +40,7 @@ for commit in $(git log --format="%H" ${old_ref}..${ref}); do
   changed_files=$(git diff-tree --no-commit-id --name-only -r $commit)
 
   for file in ${changed_files[@]}; do
-    if [[ " ${app_src_files[*]} " == *"$file"* ]]; then
+    if printf '%s\n' "${app_src_files[@]}" | grep -Fxq "$file"; then
       relevant_commits+=($commit)
       break
     fi

diff --git a/ci/core/tasks/open-charts-pr.sh b/ci/core/tasks/open-charts-pr.sh
@@ -40,7 +40,7 @@ for commit in $(git log --format="%H" ${old_ref}..${ref}); do
   changed_files=$(git diff-tree --no-commit-id --name-only -r $commit)
 
   for file in ${changed_files[@]}; do
-    if [[ " ${app_src_files[*]} " == *"$file"* ]]; then
+    if printf '%s\n' "${app_src_files[@]}" | grep -Fxq "$file"; then
       relevant_commits+=($commit)
       break
     fi

diff --git a/ci/tasks/open-charts-pr.sh b/ci/tasks/open-charts-pr.sh
@@ -41,7 +41,7 @@ for commit in $(git log --format="%H" ${old_ref}..${ref}); do
   changed_files=$(git diff-tree --no-commit-id --name-only -r $commit)
 
   for file in ${changed_files[@]}; do
-    if [[ " ${app_src_files[*]} " == *"$file"* ]]; then
+    if printf '%s\n' "${app_src_files[@]}" | grep -Fxq "$file"; then
       relevant_commits+=($commit)
       break
     fi