Skip to content

Commit 7945d60

Browse files
committed
Remove redundant github.repository check from both jobs
The `pull_request_target` trigger always runs in the context of the base repo, so `github.repository` is always `integrations/terraform-provider-github` for PRs opened against this repo. For issues, the workflow only fires on the repo's own issue tracker. The check adds no value in either case. Addresses review feedback from @stevehipwell.
1 parent 82aa151 commit 7945d60

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

.github/workflows/immediate-response.yaml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,7 @@ jobs:
2121
github.actor != 'dependabot[bot]' &&
2222
github.actor != 'renovate[bot]' &&
2323
github.actor != 'github-actions[bot]' &&
24-
github.actor != 'octokitbot' &&
25-
github.repository == 'integrations/terraform-provider-github'
24+
github.actor != 'octokitbot'
2625
runs-on: ubuntu-latest
2726
permissions:
2827
issues: write
@@ -54,8 +53,7 @@ jobs:
5453
github.actor != 'dependabot[bot]' &&
5554
github.actor != 'renovate[bot]' &&
5655
github.actor != 'github-actions[bot]' &&
57-
github.actor != 'octokitbot' &&
58-
github.repository == 'integrations/terraform-provider-github'
56+
github.actor != 'octokitbot'
5957
runs-on: ubuntu-latest
6058
permissions:
6159
issues: write

0 commit comments

Comments
 (0)