Make the AddLabel workflow work as expected

 Make AddLabel workflow in chain of "MaxText Package Tests".
 Remove single commit enforcement, as it is not in practice
 Make AddLabel workflow exit(1) to show failed if it fails to add label.

Author: Charles Li

.github/workflows/AddLabel.yml

@@ -16,7 +16,7 @@ name: Add Label
 
 on:
   workflow_run:
-    workflows: [Tests, CodeQL]
+    workflows: ["MaxText Package Tests"]
     types:
       - completed
   pull_request_review:
@@ -31,7 +31,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/github-script@v7
+      - name: Add Pull Request Label
+        uses: actions/github-script@v7
         with:
           script: |
             const owner = "google"
@@ -42,7 +43,7 @@ jobs:
             } else if (context.payload.workflow_run !== undefined) {
               if (context.payload.workflow_run.pull_requests.length === 0) {
                 console.log("This workflow is NOT running within a PR's context")
-                process.exit()
+                process.exit(1)
               }
               console.log(context.payload.workflow_run.pull_requests)
               pull_number = context.payload.workflow_run.pull_requests[0].number
@@ -68,15 +69,15 @@ jobs:
 
             if (reviews.data.length === 0) {
               console.log("Not adding pull ready because the PR is not approved yet.")
-              process.exit()
+              process.exit(1)
             }
             let is_approved = false
             if (decision_result.repository.pullRequest.reviewDecision === "APPROVED") {
               is_approved = true
             }
             if (!is_approved) {
               console.log("Not adding pull ready because the PR is not approved yet by sufficient code owners.")
-              process.exit()
+              process.exit(1)
             }
 
             const commits = await github.rest.pulls.listCommits({
@@ -85,11 +86,11 @@ jobs:
               pull_number,
               per_page: 100,
             })
-            // Check that the number of commits in the PR is 1.
-            if (commits.data.length !== 1) {
-              console.log("Not adding pull ready because the PR has more than one commit. Please squash your commits.")
-              process.exit(1)
-            }
+            // Check that the number of commits in the PR is 1. This is not enforced in practice.
+            // if (commits.data.length !== 1) {
+            //  console.log("Not adding pull ready because the PR has more than one commit. Please squash your commits.")
+            //  process.exit(1)
+            // }
             const ref = commits.data.slice(-1)[0].sha
             const checkRuns = await github.rest.checks.listForRef({
               owner,
@@ -98,13 +99,13 @@ jobs:
             })
             if (checkRuns.data.check_runs.length === 0) {
               console.log("Not adding pull ready because no check runs are associated with the last commit: " + ref)
-              process.exit()
+              process.exit(1)
             }
             for (const checkRun of checkRuns.data.check_runs) {
               if (checkRun.name.endsWith(context.job)) continue
               if (checkRun.conclusion !== "success") {
                 console.log("Not adding pull ready because " + checkRun.name + " has not passed yet: " + checkRun.html_url)
-                process.exit()
+                process.exit(1)
               }
             }
             console.log("Adding pull ready label because the PR is approved AND all the check runs have passed")