Merge pull request #3239 from Azure/anthony-c-martin-patch-2

Set exit code on failure
azure-sdk · Oct 17, 2024 · 839d3b9 · 839d3b9
2 parents b752c08 + 3885574
commit 839d3b9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.github/workflows/generate-schemas.yml b/.github/workflows/generate-schemas.yml
@@ -57,6 +57,7 @@ jobs:
 
       - name: Run generator
         run: |
+          set -e
           rm -Rf "$GITHUB_WORKSPACE/schemas"
           rm -Rf "$GITHUB_WORKSPACE/summary.log"
           mkdir -p "$GITHUB_WORKSPACE/schemas"
@@ -69,6 +70,9 @@ jobs:
             --combine-batch-mode true
         working-directory: generator
 
+      - name: Print summary
+        run: cat summary.log
+
       - name: Upload Schemas
         uses: actions/upload-artifact@v4
         with:
@@ -125,4 +129,4 @@ jobs:
           commit_message: Update Generated Schemas
           branch: autogenerate-batch
           push_options: '--force'
-          create_branch: true
+          create_branch: true
diff --git a/generator/cmd/generateall.ts b/generator/cmd/generateall.ts
@@ -104,7 +104,9 @@ executeSynchronous(async () => {
                     pkg.result = 'failed';
                     console.log(colors.red(`Caught exception processing autogenlist entry ${autoGenConfig.basePath}.`));
                     console.log(colors.red(`${(error as Error)?.stack || error}`));
-
+
+                    // return a non-zero exit code to mark the process as failed
+                    process.exitCode = 1;
                     // Use markdown formatting as this summary will be included in the PR description
                     logOut(summaryLogger, 
                         `<details>
@@ -118,6 +120,8 @@ executeSynchronous(async () => {
                 packages.push(pkg);
             }
         } catch (error) {
+            // return a non-zero exit code to mark the process as failed
+            process.exitCode = 1;
             // Use markdown formatting as this summary will be included in the PR description
             // This error usually indicates that a file has not been found (readme)
             logOut(summaryLogger,