Skip to content

Commit 14be6eb

Browse files
authored
Fix double execution of CI workflow on initial PR push (#317)
1 parent 7ad6aee commit 14be6eb

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
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: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
# Unreleased
22

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.
8+
39
## 🔩 Internal
410
* Relocked dependencies
5-
* Update referenced github actions
11+
* Update referenced github actions

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)