Skip to content

Handle re-opening of PRs under community reviews assignment - #2494

Merged
Urgau merged 1 commit into
rust-lang:mainfrom
Urgau:assign-community-review-reopened
Aug 28, 2026
Merged

Handle re-opening of PRs under community reviews assignment#2494
Urgau merged 1 commit into
rust-lang:mainfrom
Urgau:assign-community-review-reopened

Conversation

@Urgau

@Urgau Urgau commented Aug 27, 2026

Copy link
Copy Markdown
Member

When PRs under community reviews assignment are closed we remove the label (eg. S-waiting-on-community-reviews), which is expected, but we didn't added it back when the PR was being reopened, causing the PR to never get a assignee after reaching the desired minimum community reviews.

This PR therefore adds the plumbing to re-add the label when the PR is being re-opened.

Reported at rust-lang/rust-clippy#17634 (comment)

@Urgau

Urgau commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

Hum, there is technically the case of the assignee un-assigning themselves from the PR before it's re-opened, in that case there is enough approvals, but I don't know what to do in that case.

The current logic in the PR only adds the label and doesn't not assign someone, same as with normal PRs. I guess we'll figure-out if we ever have this case.

@Kobzol Kobzol left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Indeed, one at a time. Thanks!

View changes since this review

Comment thread src/handlers/assign.rs Outdated
@Urgau
Urgau force-pushed the assign-community-review-reopened branch from cdaa6d3 to ba939ee Compare August 28, 2026 16:21
@Urgau
Urgau enabled auto-merge August 28, 2026 16:22
@Urgau
Urgau added this pull request to the merge queue Aug 28, 2026
Merged via the queue into rust-lang:main with commit 0d7ad16 Aug 28, 2026
4 checks passed
@Urgau
Urgau deleted the assign-community-review-reopened branch August 28, 2026 16:30
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