Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

actions: use GitHub PAT with project write permissions #92405

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

rail
Copy link
Member

@rail rail commented Nov 23, 2022

Previously, the default token had no access to write to projects.

This PR changes the action to use the token with proper permissions set.

Release note: None
Epic: None

Previously, the default token had no access to write to projects.

This PR changes the action to use the token with proper permissions set.

Release note: None
@rail rail added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-dev-inf A-ci Continuous Integration labels Nov 23, 2022
@rail rail requested a review from ajstorm November 23, 2022 18:30
@rail rail self-assigned this Nov 23, 2022
@rail rail marked this pull request as ready for review November 23, 2022 18:30
@rail rail requested a review from a team as a code owner November 23, 2022 18:30
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rail
Copy link
Member Author

rail commented Nov 23, 2022

bors r=ajstorm

@craig
Copy link
Contributor

craig bot commented Nov 23, 2022

Build succeeded:

@craig craig bot merged commit 5173d51 into cockroachdb:master Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Continuous Integration C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-dev-inf
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants