Skip to content

Change this token name to make it more specific to its usage#31

Merged
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:change-the-org-membership-check-token-name
Feb 9, 2026
Merged

Change this token name to make it more specific to its usage#31
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:change-the-org-membership-check-token-name

Conversation

@derekmisler
Copy link
Contributor

@derekmisler derekmisler commented Feb 5, 2026

Closes https://github.com/docker/gordon/issues/98

Summary

Renames the ORG_MEMBERSHIP_TOKEN secret to CAGENT_ORG_MEMBERSHIP_TOKEN to make its purpose more explicit and avoid naming conflicts with other GitHub Actions workflows that might use generic org membership tokens.

Changes

  • .github/workflows/review-pr.yml: Updated secret name in workflow inputs, GitHub script token reference, and error messages
  • review-pr/README.md: Updated documentation to reflect the new secret name

Breaking Changes

Yes - Existing workflows using this action must rename their secret from ORG_MEMBERSHIP_TOKEN to CAGENT_ORG_MEMBERSHIP_TOKEN. The workflow will fail with a clear error message if the old secret name is still in use.

How to Test

  • Verify workflows using this action still function correctly after renaming the secret
  • Test that the error message displays correctly when the secret is missing or invalid
  • Confirm auto-review functionality works for org members with the new secret name

@derekmisler derekmisler self-assigned this Feb 5, 2026
@derekmisler
Copy link
Contributor Author

/describe

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

✅ PR description has been generated and updated!

@derekmisler derekmisler marked this pull request as ready for review February 5, 2026 14:07
@derekmisler derekmisler requested a review from a team as a code owner February 5, 2026 14:07
@derekmisler derekmisler merged commit 388f055 into docker:main Feb 9, 2026
8 checks passed
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.

2 participants