Phabricator/Bugzilla Equivalents: Difference between revisions

no edit summary
No edit summary
 
Line 59: Line 59:


== Request '''checkin-needed''' to land your changes ==
== Request '''checkin-needed''' to land your changes ==
At this time '''checkin-needed''' requests should use the current [https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch#Getting_the_patch_checked_into_the_tree checkin-needed process]. '''checkin-needed''' can be set on the bug linked to the Phabricator revision you wish to land. There is a plan to move '''checkin-needed''' to Phabricator fully, look out for an announcement when that is further along (Tracking [https://bugzilla.mozilla.org/show_bug.cgi?id=1514807 Bug 1514807]).
https://groups.google.com/g/firefox-dev/c/3JGrgQMM6lM
 
We intend to phase out the “Check-in Needed” flag. Instead, reviewers should queue patches in Lando once they are satisfied with the submitted patch.
 
In order to help ensure that nothing slips through the cracks, the existing autonag bot will be updated to needinfo not only the patch author but also the reviewer if it has gotten r+ but not landed within 2 weeks. The patch contribution documentation will also be updated to take into account these changes.


= Responding to requests =
= Responding to requests =
Confirmed users
1,198

edits