Skip to content

Commit 0d0154b

Browse files
trallardxuanxu
andauthored
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]>
1 parent 4eb379d commit 0d0154b

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/CI.yml

+2
Original file line numberDiff line numberDiff line change
@@ -228,6 +228,8 @@ jobs:
228228
- name: "Coverage comment 💬"
229229
uses: py-cov-action/python-coverage-comment-action@v3
230230
id: coverage_comment
231+
# avoid running this on schedule or releases
232+
if: github.event.workflow_run.event != 'schedule' && github.event.workflow_run.event != 'release'
231233
with:
232234
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
233235

.github/workflows/publish.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ permissions:
1717
contents: read
1818

1919
jobs:
20-
# calls our tests workflow
20+
# calls our general CI workflow (tests, build docs, etc.)
2121
tests:
2222
uses: ./.github/workflows/CI.yml
2323

0 commit comments

Comments
 (0)