← 输入框|github.com|刷新
Skip to content

gh-1359: Document follow-up for out-of-scope review suggestions - #1940

Open
badrisatyam1-ctrl wants to merge 2 commits into
python:mainfrom
badrisatyam1-ctrl:codex/follow-up-review-comments
Open

badrisatyam1-ctrl wants to merge 2 commits into
python:mainfrom
badrisatyam1-ctrl:codex/follow-up-review-comments

Conversation

@badrisatyam1-ctrl

Copy link
Copy Markdown

Closes #1359\n\nAdd concise guidance for tracking a concrete review suggestion that is outside the current PR scope: agree who will track it, then create an issue or PR linking to the original comment. Clarify that opening an issue is a reminder and context, not a commitment to implement the follow-up.\n\nValidation: git diff --check passed. No tests were run for this documentation-only change.\n\nAI disclosure: OpenAI Codex assisted with research and drafting; the text was checked against the issue discussion and current guide.

@python-cla-bot

python-cla-bot Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@read-the-docs-community

read-the-docs-community Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

@StanFromIreland

Copy link
Copy Markdown
Member

Please sign the CLA, we cannot review otherwise.

@StanFromIreland
StanFromIreland marked this pull request as draft October 27, 2026 12:51
@badrisatyam1-ctrl
badrisatyam1-ctrl marked this pull request as ready for review October 27, 2026 15:31
@hugovk

hugovk commented Oct 28, 2026

Copy link
Copy Markdown
Member

@badrisatyam1-ctrl It would be helpful if you can limit the number of open PRs to something like three. Opening a lot in a short amount of time makes it difficult for us to review. You can keep these open, but please don't open more.

Also, please don't needlessly update your branch from main, it pings all the maintainers each and every time. You can do it to resolve conflicts, or if the CI hasn't run for a long time (like months) and you want to make sure it's still passes.

Finally, make sure you read the AI policy. It's important you have read and understand the changes in your PRs, and that you've not just let your LLM create them on autopilot. I don't want to be the first human to read "your" text. It definitely looks like you've not read this PR, because the description contains "\n" chars:

image

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.

Define and document a procedure for following up on reviews

3 participants