-
Notifications
You must be signed in to change notification settings - Fork 877
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
Add write permissions to PR labelling action #25356
Conversation
@kisantia can I merge this change in directly, or should I make a PR for it in main first? |
Pull Request Test Coverage Report for Build 7980817189Details
💛 - Coveralls |
can you change the target branch for this PR to main, then port it to the release branch? So that this will be fixed for the next releases too |
Done |
I'll skip the validation checks, since this only applied to the one action |
* Add null checks for undefined include/exclude patterns in search view (#25336) (#25354) * Add write permissions to PR labelling action (#25356) (#25359) * Updating STS new version (#25357) * vbump sql projects after insiders release to 1.4.2 * vbump sql projects after insiders release to 1.4.2 --------- Co-authored-by: Cory Rivera <corivera@microsoft.com>
Opening this PR to see if the permissions issue is fixed with the PR labelling action.
Edit: Label action succeeded, so we can actually merge this change