From 4c96dca661654ca7511a11f7626d0b677256d3a7 Mon Sep 17 00:00:00 2001
From: Alexander Samusev <41779041+alvicsam@users.noreply.github.com>
Date: Tue, 13 Feb 2024 11:01:44 +0100
Subject: [PATCH] [ci] Fix gitspiegel trigger (#3297)
PR removes `pull_request_target` from gitspiegel trigger because it
breaks the logic. With `pull_request_target` the action runs in any case
even for first-time contributors.
cc @mutantcornholio
---
.github/workflows/gitspiegel-trigger.yml | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/.github/workflows/gitspiegel-trigger.yml b/.github/workflows/gitspiegel-trigger.yml
index b338f7a3f62..01058ad74d0 100644
--- a/.github/workflows/gitspiegel-trigger.yml
+++ b/.github/workflows/gitspiegel-trigger.yml
@@ -13,14 +13,15 @@ on:
- unlocked
- ready_for_review
- reopened
+ # doesn't work as intended, triggers "workflow_run" webhook in any case
# the job doesn't check out any code, so it is relatively safe to run it on any event
- pull_request_target:
- types:
- - opened
- - synchronize
- - unlocked
- - ready_for_review
- - reopened
+ # pull_request_target:
+ # types:
+ # - opened
+ # - synchronize
+ # - unlocked
+ # - ready_for_review
+ # - reopened
merge_group:
# drop all permissions for GITHUB_TOKEN
--
GitLab