BUG - Remove schedule trigger for comment action (#1914)

The `publish-to-pypi` workflow has failed due to the coverage comment action since there is no PR or separate coverage branch. This PR prevents the comment workflow from triggering in such cases and during releases to avoid potential hiccups. --------- Co-authored-by: Juanjo Bazán <[email protected]>
pydata · Jul 4, 2024 · 0d0154b · 0d0154b
1 parent 4eb379d
commit 0d0154b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -228,6 +228,8 @@ jobs:
       - name: "Coverage comment 💬"
         uses: py-cov-action/python-coverage-comment-action@v3
         id: coverage_comment
+        # avoid running this on schedule or releases
+        if: github.event.workflow_run.event != 'schedule' && github.event.workflow_run.event != 'release'
         with:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 

diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -17,7 +17,7 @@ permissions:
   contents: read
 
 jobs:
-  # calls our tests workflow
+  # calls our general CI workflow (tests, build docs, etc.)
   tests:
     uses: ./.github/workflows/CI.yml