<div dir="ltr"><div>It is, unfortunately, unlikely that there will be "first class" support for commit series before MozReview is decommissioned, which is mainly driven by the pending exit of the data center in which it currently lives.<br></div><div><br></div><div>It's a difficult feature to get right, especially when supporting confidential patches. We have started a conversation with Phacility about better upstream support for commit series, and we have a few prototypes of our own solutions. It's also worth noting that such support is at least better than Bugzilla (interestingly, commit series as attachments are still popular there). See <a href="http://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#series-of-commits">http://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#series-of-commits</a> for the current state.</div><div><br></div><div>
 I realize this is not ideal; my team also often works with series of multiple commits, though they've adapted to Phabricator (see, for example, <a href="https://smacleod.ca/posts/commit-series-with-phabricator/">https://smacleod.ca/posts/commit-series-with-phabricator/</a>, which is also linked in the docs referenced above). This is a trade-off we made in order to reduce our maintenance burden by using third-party software where possible, with limited customization, which in turn allows us to focus more on automation specific to Firefox development that has no off-the-shelf solution at all.<br></div><div><br></div><div>Mark<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 6, 2018 at 1:54 PM, Markus Stange <span dir="ltr"><<a href="mailto:mstange@themasta.com" target="_blank">mstange@themasta.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Is there a list of features which are planned to be implemented in Phabricator + Lando before MozReview is sunset? More specifically, is Phabricator expected to have first class support of the "commit series" development workflow by the time this happens?</div><div>This is a feature that I personally would miss a lot. Of the 520 patches I've landed within the last two years, only <span class="m_-1601760347719654333gmail-message-body-wrapper"><span class="m_-1601760347719654333gmail-message-flex-body"><span class="m_-1601760347719654333gmail-message-body m_-1601760347719654333gmail-devtools-monospace"><span class="m_-1601760347719654333gmail-objectBox m_-1601760347719654333gmail-objectBox-number">132</span></span></span></span> patches were from single-patch bugs. 262 patches landed as part of a 5-commits-or-more series.</div><div><br></div><div>Thanks,</div><div>Markus<br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, Jun 6, 2018 at 10:57 AM, Mark Côté <span dir="ltr"><<a href="mailto:mcote@mozilla.com" target="_blank">mcote@mozilla.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">The Engineering Workflow team is happy to announce the release of Phabricator and Lando for general use. Going forward, Phabricator will be the primary code-review tool for modifications to the mozilla-central repository, replacing both MozReview and Splinter. Lando is an all-new automatic-landing system that works with Phabricator. This represents about a year of work integrating Phabricator with our systems and building out Lando. Phabricator has been in use by a few teams since last year, and Lando has been used by the Engineering Workflow team for several weeks and lately has successfully landed a few changesets to mozilla-central.<br><br>Phabricator is a suite of applications, but we are primarily using the code-review tool, called Differential, which will be taking the place of MozReview and Splinter. Bug tracking will continue to be done with Bugzilla, which is integrated with Phabricator. You will log into Phabricator via Bugzilla. We will soon begin sunsetting MozReview, and Splinter will be made read-only (or replaced with another patch viewer). An upcoming post will outline the plans for the deprecation, archival, and decommission of MozReview, with Splinter to follow.<br><br>I also want to thank Phacility, the company behind Phabricator, who provided both excellent support and work on Phabricator itself to meet our requirements in an exceptionally helpful and responsive way.<br><br>User documentation on Phabricator catered specifically to Mozillians can be found at <a href="https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html" target="_blank">https://moz-conduit.readthedoc<wbr>s.io/en/latest/phabricator-<wbr>user.html</a>. It is also linked from within Phabricator, in the left-hand menu on the home page.<br><br>User documentation on Lando can be found at <a href="https://moz-conduit.readthedocs.io/en/latest/lando-user.html" target="_blank">https://moz-conduit.readthedoc<wbr>s.io/en/latest/lando-user.html</a><wbr>.<br><br>MDN documentation is currently being updated.<br><br>At the moment, Phabricator can support confidential revisions when they are associated with a confidential bug, that is, a bug with one or more security groups applied. Lando, however, cannot currently land these revisions. This is a limitation we plan to fix in Q3. You can follow <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1443704" target="_blank">https://bugzilla.mozilla.org/s<wbr>how_bug.cgi?id=1443704</a> for developments. See <a href="http://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#landing-patches" target="_blank">http://moz-conduit.readthedocs<wbr>.io/en/latest/phabricator-<wbr>user.html#landing-patches</a> for our recommendations on landing patches in Phabricator without Lando.<br><br>Similarly, there are two other features which are not part of initial launch but will follow in subsequent releases:<br>* Stacked revisions. If you have a stack of revisions, that is, two or more revisions with parent-child relationships, Lando cannot land them all at once.  You will need to individually land them. This is filed as <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1457525" target="_blank">https://bugzilla.mozilla.org/s<wbr>how_bug.cgi?id=1457525</a>. <br>* Try support. Users will have to push to the Try server manually until this is implemented. See <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1466275" target="_blank">https://bugzilla.mozilla.org/s<wbr>how_bug.cgi?id=1466275</a>. <br><br>Finally, we realize there are a few oddities with the UI that we will also be fixing in parallel with the new features. See <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1466120" target="_blank">https://bugzilla.mozilla.org/s<wbr>how_bug.cgi?id=1466120</a>.<br><br>The documentation lists several ways of getting in touch with the Engineering Workflow team, but #phabricator and #lando on IRC are good starting points.<br><br></div>
<br></div></div><span class="">______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org" target="_blank">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listi<wbr>nfo/firefox-dev</a><br>
<br></span></blockquote></div><br></div>
</blockquote></div><br></div>