Skip to content

ci: support per-user PAT for staging PR authorship#4218

Merged
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:staging-pr-per-user-pat
May 10, 2026
Merged

ci: support per-user PAT for staging PR authorship#4218
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:staging-pr-per-user-pat

Conversation

@maliberty
Copy link
Copy Markdown
Member

The staging PR opened by github-actions-on-label-create.yml is currently authored by the bot account behind STAGING_GITHUB_TOKEN, hiding the real contributor on the staging side. Add a token-resolution step that, when the PR author has stored a personal access token as repository secret PAT_ (uppercase, '-' -> '_'), uses it to call the PR-creation API so the staging PR is authored by them. Falls back to the bot token when no per-user PAT is configured, preserving today's behavior.

The staging PR opened by github-actions-on-label-create.yml is currently
authored by the bot account behind STAGING_GITHUB_TOKEN, hiding the real
contributor on the staging side. Add a token-resolution step that, when
the PR author has stored a personal access token as repository secret
PAT_<LOGIN> (uppercase, '-' -> '_'), uses it to call the PR-creation API
so the staging PR is authored by them. Falls back to the bot token when
no per-user PAT is configured, preserving today's behavior.

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
@maliberty maliberty self-assigned this May 10, 2026
@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@maliberty maliberty merged commit 9e8fbc5 into The-OpenROAD-Project:master May 10, 2026
6 of 8 checks passed
@maliberty maliberty deleted the staging-pr-per-user-pat branch May 10, 2026 03:14
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