<div dir="ltr">From the docs, the bot supports<div>* rebased/merged for single commit PR</div><div>* squashed/merged for multi-commits PR</div><div>(I believe squashed would work for single commit too)</div><div><br></div><div>Matthias, are you saying we should just add -m1 here <a href="https://github.com/Gaurav0/backport/blob/master/src/backport.ts#L107">https://github.com/Gaurav0/backport/blob/master/src/backport.ts#L107</a></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 4 mars 2020 à 08:19, Matthias Kuhn <<a href="mailto:matthias@opengis.ch">matthias@opengis.ch</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
On 3/4/20 8:04 AM, Nyall Dawson wrote:<br>
> On Wed, 4 Mar 2020 at 17:01, Matthias Kuhn <<a href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a>> wrote:<br>
>> On 3/4/20 7:48 AM, Denis Rouzaud wrote:<br>
>><br>
>><br>
>><br>
>> Le mer. 4 mars 2020 à 02:56, Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>> a écrit :<br>
>>> On Wed, 4 Mar 2020 at 10:48, William Kyngesburye <<a href="mailto:woklist@kyngchaos.com" target="_blank">woklist@kyngchaos.com</a>> wrote:<br>
>>><br>
>>>> If it's just a buggy backport bot, then that's bad timing on my part, after all my dwindling spare time it took me to get caught up on my Mac build, I get this.<br>
>>> It is. The bot got seriously crippled by an upstream change, and now<br>
>>> it's a coin toss on the results you'll get.<br>
>><br>
>> I plan to give another approach a try during the HF.<br>
>> The idea would be to use a CLI tool (<a href="https://github.com/sqren/backport" rel="noreferrer" target="_blank">https://github.com/sqren/backport</a> or <a href="https://github.com/chainer/backport" rel="noreferrer" target="_blank">https://github.com/chainer/backport</a>) which would be run a cron workflow.<br>
>> Using a cron will avoid the issues of PR from forks (the workflow user has rights to create branches).<br>
>> Hopefully these tools work better than the current bot.<br>
>> The only drawback I see up to now is that you'll have to wait for the cron to start (30min?).<br>
>><br>
>><br>
>> It looks like the bot fails on cherry-picking merge commits and would (likely) do fine if a `-m` option was provided - at least that's what's missing from the commands in the comment.<br>
>><br>
>> Maybe it's as easy as adding this?<br>
> it still depends on "merges" instead of rebases... and I'm not a fan<br>
> of merge commits in general. They really make the git history messy<br>
> and difficult to read...<br>
><br>
> Nyall<br>
<br>
Then it was too good to be true. I miss the old backport bot.<br>
<br>
I am generally a fan of merges, mostly because they provide a missing <br>
link from "a set of commits" to a "consolidated feature" aka pull <br>
request with discussion. We don't have that with rebase merges I think?<br>
<br>
Matthias<br>
<br>
</blockquote></div>