Skip to content

Use GitHub App client ID for project sync token#188

Merged
zackees merged 1 commit intomainfrom
fix-project-app-token-client-id
Apr 22, 2026
Merged

Use GitHub App client ID for project sync token#188
zackees merged 1 commit intomainfrom
fix-project-app-token-client-id

Conversation

@zackees
Copy link
Copy Markdown
Member

@zackees zackees commented Apr 22, 2026

Summary

  • update add-to-project workflow to use actions/create-github-app-token@v3
  • switch token generation from legacy PROJECT_APP_ID to PROJECT_APP_CLIENT_ID
  • add the matching PROJECT_APP_CLIENT_ID repo Actions variable

Notes

This does not install the GitHub App. The FastLED-owned app with client ID Iv23liL4dLxjYFwTNWKt still needs to be installed on the FastLED org for token generation to succeed.

Related: FastLED/FastLED#2383

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow configuration for project automation.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: edb8e9cc-0ee7-4c8a-9898-8d05f4de3dd9

📥 Commits

Reviewing files that changed from the base of the PR and between b85edbc and 82143d3.

📒 Files selected for processing (1)
  • .github/workflows/add-to-project.yml

📝 Walkthrough

Walkthrough

The workflow configuration was updated to align with newer GitHub App token generation practices. The actions/create-github-app-token action was upgraded from v1 to v3, and the corresponding input parameter was changed from app-id to client-id with a new variable reference.

Changes

Cohort / File(s) Summary
GitHub Workflow Update
.github/workflows/add-to-project.yml
Updated conditional guard and token-generation step to use vars.PROJECT_APP_CLIENT_ID instead of vars.PROJECT_APP_ID, and upgraded action from v1 to v3 with corresponding input parameter rename from app-id to client-id.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Poem

🐰 A workflow hops to version three,
With CLIENT_ID, now proud and free,
Old app-id tokens fade away,
Fresh GitHub Actions save the day! 🚀

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: updating the workflow to use GitHub App client ID instead of app ID for token generation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-project-app-token-client-id

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@zackees zackees merged commit e12252b into main Apr 22, 2026
76 of 77 checks passed
@zackees zackees deleted the fix-project-app-token-client-id branch April 22, 2026 19:22
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