Skip to content

docs: ask reviewers what the diff already answers - #854

Merged
josephfusco merged 1 commit into
mainfrom
docs/review-question
Oct 16, 2026
Merged

josephfusco merged 1 commit into
mainfrom
docs/review-question

Conversation

@josephfusco

@josephfusco josephfusco commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

The reviewer question sits beside the author one at the top of the template, so both read as one ask.

Use of AI Tools

AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5.5
Used for: Assisting with the template wording.

@josephfusco josephfusco changed the title docs: ask reviewers whether a comment needed the diff docs: add a template for review comments Oct 16, 2026
@josephfusco
josephfusco marked this pull request as ready for review October 16, 2026 21:31
@github-actions

Copy link
Copy Markdown
Contributor

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props joefusco.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@josephfusco josephfusco changed the title docs: add a template for review comments docs: ask reviewers what the diff already answers Oct 16, 2026
@josephfusco
josephfusco marked this pull request as draft October 16, 2026 21:34
@josephfusco
josephfusco marked this pull request as ready for review October 16, 2026 21:36
@josephfusco
josephfusco merged commit e08f3f5 into main Oct 16, 2026
24 checks passed
@josephfusco
josephfusco deleted the docs/review-question branch October 16, 2026 21:36
@github-actions github-actions Bot added this to the 0.18.2 milestone Oct 16, 2026
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.

1 participant