mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Add a note about PR approvals [ci skip]
Sometimes folks get "Approved" reviews from other contributors who do not have merge access. This can come across as an official endorsement, especially in cases where the approval comes from someone who is a member of the Rails org, but is not on the core or committer team (me, for example). This paragraph is meant to document both that "Approved" is not the same as an official endorsement, and that we prefer people don't mark PRs as "Approved" in the first place.
This commit is contained in:
parent
364c2156af
commit
dbb3ca338e
1 changed files with 5 additions and 0 deletions
|
@ -621,6 +621,11 @@ While you're waiting for feedback on your pull request, open up a few other
|
||||||
pull requests and give someone else some! I'm sure they'll appreciate it in
|
pull requests and give someone else some! I'm sure they'll appreciate it in
|
||||||
the same way that you appreciate feedback on your patches.
|
the same way that you appreciate feedback on your patches.
|
||||||
|
|
||||||
|
Note that your pull request may be marked as "Approved" by somebody who does not
|
||||||
|
have access to merge it. Further changes may still be required before members of
|
||||||
|
the core or committer teams accept it. To prevent confusion, when giving
|
||||||
|
feedback on someone else's pull request please avoid marking it as "Approved."
|
||||||
|
|
||||||
### Iterate as Necessary
|
### Iterate as Necessary
|
||||||
|
|
||||||
It's entirely possible that the feedback you get will suggest changes. Don't get discouraged: the whole point of contributing to an active open source project is to tap into the knowledge of the community. If people are encouraging you to tweak your code, then it's worth making the tweaks and resubmitting. If the feedback is that your code doesn't belong in the core, you might still think about releasing it as a gem.
|
It's entirely possible that the feedback you get will suggest changes. Don't get discouraged: the whole point of contributing to an active open source project is to tap into the knowledge of the community. If people are encouraging you to tweak your code, then it's worth making the tweaks and resubmitting. If the feedback is that your code doesn't belong in the core, you might still think about releasing it as a gem.
|
||||||
|
|
Loading…
Reference in a new issue