139
edits
(Adding process flow summary) |
(Add Reviews section summary) |
||
| Line 44: | Line 44: | ||
=====Reviews===== | =====Reviews===== | ||
Phabricator is used for reviews, just as it would be for non-merge work. Use the bug # for the main WebRTC merge revision ([https://bugzilla.mozilla.org/show_bug.cgi?id=1654112 1654112]). Consider making all reviewers blocking to avoid issues with change sets languishing due to lack of visibility after a single reviewer has accepted a patch. | |||
=====Finalizing a Patch===== | =====Finalizing a Patch===== | ||
=====Shortcomings and Criticism===== | =====Shortcomings and Criticism===== | ||
* The user Mercurial repositories are non-publishing which leaves pushed commits in a draft state. | * The user Mercurial repositories are non-publishing which leaves pushed commits in a draft state. | ||
edits