Skip to content

Commit 92d5139

Browse files
committed
Fix double execution of CI workflow on initial PR push
1 parent 50d4f43 commit 92d5139

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

.github/workflows/ci.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ on:
77
- "gh-pages/*"
88
- "main"
99
- "master"
10-
pull_request:
11-
types: [opened, reopened]
1210
schedule:
1311
# “At 00:00 on every 7th day-of-month from 1 through 31.” (https://crontab.guru)
1412
- cron: "0 0 1/7 * *"

doc/changes/unreleased.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,7 @@
11
# Unreleased
2+
3+
## 🐞 Fixed
4+
* Fixed an issue in the CI workflow that caused it to be executed twice on the initial push of a PR if the PR branch was on the repo itself.
5+
6+
🚨 Attention: Due to these changes, the workflows will no longer be executed if the PR comes from a branch not located in this repository.
7+
As third-party contributions from outside forks are rare to nearly non-existent, this downside was considered a reasonable trade-off at this time.

exasol/toolbox/templates/github/workflows/ci.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ on:
77
- "gh-pages/*"
88
- "main"
99
- "master"
10-
pull_request:
11-
types: [opened, reopened]
1210
schedule:
1311
# “At 00:00 on every 7th day-of-month from 1 through 31.” (https://crontab.guru)
1412
- cron: "0 0 1/7 * *"

0 commit comments

Comments
 (0)