Skip to content

Commit bf1935c

Browse files
Merge pull request #19 from kernel/hypeship/sync-breakglass-from-infra
Sync the breakglass workflow with kernel/infra main
2 parents c39b889 + 095a8eb commit bf1935c

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

‎.github/workflows/breakglass-merge.yml‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,9 @@ jobs:
5151
app-id: ${{ secrets.BREAKGLASS_APP_ID }}
5252
private-key: ${{ secrets.BREAKGLASS_APP_PRIVATE_KEY }}
5353
owner: ${{ github.repository_owner }}
54-
# Deliberately NOT scoped with `repositories:`. Scoping the installation token to
55-
# the single repo holding the PR is tighter and was the right instinct, but it is
56-
# also the only functional difference between the configuration that provably
57-
# merged a PR past an approval rule and every configuration since, which has been
58-
# refused. Until that is understood, the token stays owner-wide. Narrowing it again
59-
# needs a test that shows the merge still works.
54+
# Scoped to the one repo holding the PR. Without this the token reaches every repo
55+
# the app is installed on, which is far more than merging this PR requires.
56+
repositories: ${{ github.event.repository.name }}
6057

6158
- name: Merge without approval
6259
env:

0 commit comments

Comments
 (0)