-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Single out docs-only PRs in review process. #5163
Conversation
Thanks @mkcor! I'm wondering whether to merge your PR as is, or to specify "non-controversial pull requests" (meaning that for a long doc PR with a lot of stuff, it is still better to have two reviews). Thoughts? |
I agree with you, @emmanuelle... It might be better not to merge this PR at all then, and keep the guidelines valid in the general case. I think it's not great to use the term "non-controversial" in a short bullet-point list, because it is subject to judgement. On the contrary, "docs-only" is objective, but we do want two reviewers if the "docs-only" PR is long and/or complex, indeed... So I would suggest using our judgement just like we've done so far. Feel free to close! |
@mkcor not completely sure here, I think core devs are not always aware (or they forget) that they can merge small doc PRs they are happy with... Let's wait for others to chime in and in any case, thanks for bringing up this point. |
I'm happy to go one way or the other here... |
Co-authored-by: Gregory R. Lee <grlee77@gmail.com>
Sorry, this almost fell in the cracks! I have included @grlee77's suggestion, which seems consensual. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating. This looks ready to me
Co-authored-by: Gregory R. Lee <grlee77@gmail.com>
Description
@grlee77 updating the guidelines as per #5120 (comment).
Checklist
./doc/examples
(new features only)./benchmarks
, if your changes aren't covered by anexisting benchmark
For reviewers
later.
__init__.py
.doc/release/release_dev.rst
.