Skip to content

ci: migrate create-github-app-token to client-id#101

Merged
zfarrell merged 2 commits into
mainfrom
fix/app-token-client-id
Jun 6, 2026
Merged

ci: migrate create-github-app-token to client-id#101
zfarrell merged 2 commits into
mainfrom
fix/app-token-client-id

Conversation

@zfarrell

@zfarrell zfarrell commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Replaces the deprecated app-id input with client-id for the hotdata-automation app, clearing the create-github-app-token deprecation warning in CI.

claude[bot]
claude Bot previously approved these changes Jun 6, 2026

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified both workflows migrate app-idclient-id consistently. Client ID is a public identifier; private key stays in secrets. LGTM.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean migration from app-id to client-id. Both workflow files updated consistently, action SHA-pinned to v3.2.0, and the private key remains in secrets. The client-id is a public identifier so hardcoding it is fine.

@zfarrell zfarrell merged commit 5aee4b4 into main Jun 6, 2026
3 checks passed
@zfarrell zfarrell deleted the fix/app-token-client-id branch June 6, 2026 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant