Media/WebRTC/libwebrtc Update Process Transition Plan: Difference between revisions

Jump to navigation Jump to search
Add Reviews section summary
(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.
139

edits

Navigation menu