Skip to content

PR label management needs PR write permission#1608

Merged
simongdavies merged 1 commit into
hyperlight-dev:mainfrom
simongdavies:review-label-management
Jul 2, 2026
Merged

PR label management needs PR write permission#1608
simongdavies merged 1 commit into
hyperlight-dev:mainfrom
simongdavies:review-label-management

Conversation

@simongdavies

@simongdavies simongdavies commented Jul 1, 2026

Copy link
Copy Markdown
Member

This may not be enough to make it always work, its still might not remove label from PRs created from forks, might still need the pull_request_target event for that

@simongdavies simongdavies added the kind/bugfix For PRs that fix bugs label Jul 1, 2026
Copilot AI review requested due to automatic review settings July 1, 2026 22:56
@simongdavies simongdavies added the ready-for-review PR is ready for (re-)review label Jul 1, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the GitHub Actions workflow that manages the ready-for-review label so it has sufficient token permissions to modify pull requests when invoking the org-wide reusable workflow.

Changes:

  • Escalates workflow token permission from pull-requests: read to pull-requests: write in .github/workflows/ready-for-review-label.yml.

Comment thread .github/workflows/ready-for-review-label.yml
@simongdavies simongdavies enabled auto-merge (squash) July 2, 2026 07:58
Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
@simongdavies simongdavies force-pushed the review-label-management branch from 28e17b2 to d4d362e Compare July 2, 2026 08:07
@simongdavies simongdavies merged commit 593e901 into hyperlight-dev:main Jul 2, 2026
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bugfix For PRs that fix bugs ready-for-review PR is ready for (re-)review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants