Update coverage job to run in the right conditions (#5183)

sqlfluff · Sep 9, 2023 · f6c0758 · f6c0758
1 parent 78eae3a
commit f6c0758
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.github/workflows/ci-tests.yml b/.github/workflows/ci-tests.yml
@@ -373,12 +373,12 @@ jobs:
         with:
           name: html-report
           path: htmlcov
-        if: ${{ failure() }}
+        if: failure() && github.event_name == 'pull_request'
 
-      - name: Update PR Comment.
+      - name: Update PR comment with coverage report.
         uses: actions/github-script@v6
         # Run even if the coverage step failed.
-        if: github.event_name == 'pull_request'
+        if: always() && github.event_name == 'pull_request'
         with:
           script: |
             // First list the existing comments