<div dir="ltr"><div>Reading through <a href="https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#series-of-commits">https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#series-of-commits</a> it seems the actions that you have to do to push a review request with a commit series is to run `arc diff` on each commit in the series, and then load up the Phabricator UI and manually associate the parent-child relationships between all commits? So if you then amend the bottom most commit and then want to resubmit, you need to repush the entire series and then re-set the parent/child relationships? That sounds rather manual and error prone, and a significant UX regression from simply typing `hg push review`.</div><div><br></div><div>The review process is critical to how we ship code, so I don't think we should accept making requesting and performing reviews harder and more error prone.</div><div><br></div><div>If the problem is that it was decided we are moving out of a datacenter, can we write a cheque to extend our lease there, or move the Phabricator machine (or an image of it) somewhere else?</div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 7, 2018 at 12:18 PM, Chris Pearce <span dir="ltr"><<a href="mailto:cpearce@mozilla.com" target="_blank">cpearce@mozilla.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>A quick `hg log` on mozilla-central shows me that the commit series workflow is extremely common. I use it myself a lot, and believe it is good practise. So I would be very disappointed if we made it inconvenient to break up work in this way.</div><div><br></div><div>I'm not trying to add stop energy to improving our tools and processes, but I feel taking a review UX regression in the short term is not a good trade off. Can we please delay obsoleting MozReview until Phabricator has better support for the commit-series workflow?<br></div><div><br></div><div><br></div><div>Regards,</div><div>Chris Pearce.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Thu, Jun 7, 2018 at 9:44 AM, Jean-Yves Avenard <span dir="ltr"><<a href="mailto:jyavenard@mozilla.com" target="_blank">jyavenard@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"><span><br>
<br>
> On 6 Jun 2018, at 7:54 pm, Markus Stange <<a href="mailto:mstange@themasta.com" target="_blank">mstange@themasta.com</a>> wrote:<br>
> <br>
> 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?<br>
> This is a feature that I personally would miss a lot. Of the 520 patches I've landed within the last two years, only 132 patches were from single-patch bugs. 262 patches landed as part of a 5-commits-or-more series.<br>
> <br>
> Thanks,<br>
> Markus<br>
<br>
</span>Wait what?<br>
<br>
No ability to submit a series of patch ???<br>
<br>
I can’t recall the last time I pushed a single patch to mozreview. 1 commit = 1 scope, that’s the way it should always be.<br>
<br>
I don’t understand how we could consider a compulsory switch to phabricator, or even discuss a switch to it without support of this essential feature.<br>
<span class="m_-7915872636847350786HOEnZb"><font color="#888888"><br>
JY<br>
<br>
<br>
</font></span><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>