aglinxinyuan · Aug 17, 2026 · Aug 17, 2026 · Aug 29, 2026 · Aug 29, 2026 · Aug 31, 2026
diff --git a/.github/workflows/automatic-email-notif-on-ddl-change.yml b/.github/workflows/automatic-email-notif-on-ddl-change.yml
@@ -40,9 +40,17 @@ jobs:
           FILE=$(git diff --name-only --diff-filter=A \
             ${{ github.event.pull_request.base.sha }} \
             ${{ github.event.pull_request.merge_commit_sha }} \
-            -- 'sql/updates/')
-          echo "sql_file=$FILE" >> $GITHUB_OUTPUT
+            -- 'sql/updates/*.sql')
+          # git diff prints one path per line, so a PR adding two migrations
+          # would put a bare second line in $GITHUB_OUTPUT -- no key=, which
+          # makes the runner reject the whole file and the step fail.
+          echo "sql_file=${FILE//$'\n'/, }" >> $GITHUB_OUTPUT
+      # A ddl-change PR need not add a migration: it may edit or delete files
+      # under sql/updates/, add a non-.sql file such as this README, or touch
+      # a .sql file elsewhere in the repo. Any of those leaves operators
+      # nothing to apply, so no instruction goes out.
       - name: Send email
+        if: steps.get_sql_file.outputs.sql_file != ''
         run: |
           curl --ssl-reqd \
             --url "smtps://smtp.gmail.com:465" \

diff --git a/sql/updates/01.sql b/sql/updates/01.sql
diff --git a/sql/updates/02.sql b/sql/updates/02.sql
diff --git a/sql/updates/03.sql b/sql/updates/03.sql
diff --git a/sql/updates/04.sql b/sql/updates/04.sql
diff --git a/sql/updates/05.sql b/sql/updates/05.sql
diff --git a/sql/updates/06.sql b/sql/updates/06.sql