websocket: autobahn workflow should fail on error (#3259)

nodejs · May 14, 2024 · 903c667 · 903c667
1 parent c8352d7
commit 903c667
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/.github/workflows/autobahn.yml b/.github/workflows/autobahn.yml
@@ -53,9 +53,11 @@ jobs:
       - name: Report into CI
         id: report-ci
         run: npm run test:websocket:autobahn:report
+        env:
+          FAIL_ON_ERROR: true
 
       - name: Generate Report for PR Comment
-        if: github.event_name == 'pull_request_target'
+        if: (failure() || success()) && github.event_name == 'pull_request_target'
         id: report-markdown
         run: |
           echo "comment<<nEOFn" >> $GITHUB_OUTPUT
@@ -65,7 +67,7 @@ jobs:
           REPORTER: markdown
 
       - name: Comment PR
-        if: github.event_name == 'pull_request_target'
+        if: (failure() || success()) && github.event_name == 'pull_request_target'
         uses: thollander/actions-comment-pull-request@v2
         with:
           message: ${{ steps.report-markdown.outputs.comment }}