Skip to content

Commit c2dc671

Browse files
authored
Delete the vestigial github-app-id input (#117)
#93 moved the token action to the client ID. The old input survived as a fallback for callers that no longer exist, and the generated caller has never passed it. It also invites the wrong credential. The name says App ID, which is the numeric value on the App settings page, but create-github-app-token needs the Iv23 client ID. An adopter reading the input list had a documented reason to supply the wrong one. CHANGELOG entries are left alone. They record what shipped in past releases and are not instructions. Closes #116
1 parent bf37758 commit c2dc671

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

.github/workflows/simplycubed.yml

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,6 @@ on:
1313
default: ""
1414
required: false
1515
type: string
16-
github-app-id:
17-
description: Deprecated, kept so an existing caller keeps working. Pass
18-
github-app-client-id instead.
19-
default: ""
20-
required: false
21-
type: string
2216
issue-number:
2317
description: Issue number to run, when triggered from issues:labeled.
2418
default: ""
@@ -80,7 +74,7 @@ jobs:
8074
id: app-token
8175
uses: actions/create-github-app-token@bcd2ba49218906704ab6c1aa796996da409d3eb1 # v3.2.0
8276
with:
83-
client-id: ${{ inputs['github-app-client-id'] || inputs['github-app-id'] }}
77+
client-id: ${{ inputs['github-app-client-id'] }}
8478
private-key: ${{ secrets['github-app-private-key'] }}
8579
owner: ${{ steps.repo.outputs.owner }}
8680
repositories: ${{ steps.repo.outputs.name }}
@@ -168,7 +162,7 @@ jobs:
168162
id: app-token
169163
uses: actions/create-github-app-token@bcd2ba49218906704ab6c1aa796996da409d3eb1 # v3.2.0
170164
with:
171-
client-id: ${{ inputs['github-app-client-id'] || inputs['github-app-id'] }}
165+
client-id: ${{ inputs['github-app-client-id'] }}
172166
private-key: ${{ secrets['github-app-private-key'] }}
173167
owner: ${{ steps.repo.outputs.owner }}
174168
repositories: ${{ steps.repo.outputs.name }}

0 commit comments

Comments
 (0)