<div dir="auto">Hi devs,<div dir="auto"><br></div><div dir="auto">See below for cool new stuff.</div><div dir="auto">I guess we also want such a Backport bot...</div><div dir="auto"><br></div><div dir="auto">Markus</div><div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">---------- Forwarded message ---------<br>From: <strong class="gmail_sendername" dir="auto">Nyall Dawson</strong> <span dir="ltr"><<a href="mailto:nyall.dawson@gmail.com">nyall.dawson@gmail.com</a>></span><br>Date: So., 17. März 2019, 20:14<br>Subject: Re: [PROJ] [gdal-dev] github backporting application<br>To: Even Rouault <<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>><br>Cc: gdal dev <<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a>>, PROJ <<a href="mailto:proj@lists.osgeo.org">proj@lists.osgeo.org</a>><br></div><br><br>On Mon, 18 Mar 2019 at 09:06, Even Rouault <<a href="mailto:even.rouault@spatialys.com" target="_blank" rel="noreferrer">even.rouault@spatialys.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> I've just seen that QGIS has started using a github backporting application,<br>
> <a href="https://github.com/apps/backporting" rel="noreferrer noreferrer" target="_blank">https://github.com/apps/backporting</a>, and I've just setup it for GDAL and PROJ<br>
><br>
> So the workflow is :<br>
> 1) Create a pull request against master<br>
> 2) If backport is needed, label it with "backport {branchname}" (I've created<br>
> a "backport release/2.4" for GDAL and "backport 6.0" for PROJ)<br>
> 3) Merge the pull request<br>
> 4) The backport application automatically creates a new pull request against<br>
> the target backport branch, and cherry-picks all the commits of the original<br>
> pull request<br>
> 5) Wait for CI to complete (or not) and merge (or not) this new PR<br>
><br>
> Of course this doesn't solve automagically conflicts etc, but can be<br>
> convenient for trivial backports when branches are close enough<br>
<br>
Just from a QGIS developer's perspective:<br>
<br>
The system is working OK, but there's two current limitations in the<br>
bot which make it difficult to work within nicely:<br>
<br>
1. Backported PRs have a generic name, e.g. "Backport #9528"<br>
(<a href="https://github.com/qgis/QGIS/pull/9531" rel="noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/9531</a>). This makes it hard to see<br>
what outstanding backports are open without diving into the details of<br>
each. There's a fix here for the bot, but not merged yet:<br>
<a href="https://github.com/tibdex/backport/pull/19" rel="noreferrer noreferrer" target="_blank">https://github.com/tibdex/backport/pull/19</a> . On a similar vein, the<br>
backported PRs don't include original author information, which in my<br>
view makes reviewing a bit harder (since we have policies in place<br>
that require a review from a different developer to the original<br>
submitter for backports to the stable branch. Not a blocker, but just<br>
opens the policy for easier abuse).<br>
<br>
2. It's hard to track multiple backports, because currently the bot<br>
only supports a single branch. See<br>
<a href="https://github.com/tibdex/backport/pull/18" rel="noreferrer noreferrer" target="_blank">https://github.com/tibdex/backport/pull/18</a> . We have to currently<br>
"step" the backports back through multiple branches individually by<br>
tagging the first auto created backport with the next branch to<br>
backport to. Hopefully Matthias' PR is merged soon and fixes this<br>
situation, but in the meantime it's fragile and potentially a backport<br>
could accidently be skipped for a branch.<br>
<br>
Nyall<br>
_______________________________________________<br>
PROJ mailing list<br>
<a href="mailto:PROJ@lists.osgeo.org" target="_blank" rel="noreferrer">PROJ@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/proj" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/proj</a><br>
</div>