Skip to content

Remove automatic review request - #6392

Open
VPS-Obi wants to merge 1 commit into
mainfrom
claude/nice-newton-gdqdg3
Open

VPS-Obi wants to merge 1 commit into
mainfrom
claude/nice-newton-gdqdg3

Conversation

@VPS-Obi

@VPS-Obi VPS-Obi commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Every pull request automatically requested a review from me, which made the request meaningless: an automated request can't tell whether a pull request is actually ready for review (the author may have forgotten to open it as a draft), so it isn't clear whether a review is wanted. Reviews should be requested explicitly instead. Pull requests will also be reviewed by the AI code reviewer soon, and since I watch the repository, nothing gets missed without the automatic request.

Assigning the author to their own pull request stays in place, and now also happens for drafts — the action skips them unless runOnDraft is enabled. Without the review request there is nothing left to do when a draft is marked ready, so the workflow only runs on opened.

Changeset

None needed — workflow only

Session: https://claude.ai/code/session_01JnTSU9CbpR8kVZF6J96Dec

@VPS-Obi VPS-Obi changed the title Remove auto-assign workflow and configuration Remove auto assign workflow Sep 17, 2026
@VPS-Obi VPS-Obi self-assigned this Sep 17, 2026
@VPS-Obi
VPS-Obi marked this pull request as ready for review September 17, 2026 09:23
@VPS-Obi
VPS-Obi requested review from VPS-thodax and nsams September 17, 2026 09:23
@nsams

nsams commented Sep 17, 2026

Copy link
Copy Markdown
Member

why remove author as assignee? I would miss that

@VPS-Obi
VPS-Obi force-pushed the claude/nice-newton-gdqdg3 branch from 4981b0d to b43cfcb Compare September 17, 2026 11:24
@VPS-Obi VPS-Obi changed the title Remove auto assign workflow Remove automatic review request Sep 17, 2026
Requesting a review should be an explicit decision by the author: an
automatic request can't tell whether a pull request is actually ready
(e.g. draft not set), and pull requests will be reviewed by the AI code
reviewer soon. Missed pull requests aren't a concern since the
repository is watched.

Assigning the author stays in place, and now also happens for drafts,
which the action skips by default. Without the review request there is
nothing left to do when a draft is marked ready, so the workflow only
runs on opened.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JnTSU9CbpR8kVZF6J96Dec
@VPS-Obi
VPS-Obi force-pushed the claude/nice-newton-gdqdg3 branch from b43cfcb to 4c50a36 Compare September 17, 2026 11:27
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.

3 participants