Web Testing/Automation/CodeReviewProcess: Difference between revisions

Line 46: Line 46:


== Who can do merges? ==
== Who can do merges? ==
* '''For starters, you *cannot* merge-in your own code/pull requests'''
* In general, you cannot merge-in your own code/pull requests, but if it has received a positive review from someone else you can merge it yourself on their behalf.
* Anyone who has tested the patch against all the affected/intended environments (and made sure it's had at least one review)
* Anyone who has tested the patch against all the affected/intended environments (and made sure it's had at least one review)
Confirmed users
425

edits