<div dir="ltr"><div>Since most seem to agree with the bot implementation, can we do it in a way that:<br>- the PR owner (and other watchers) receive a message warning that the PR is going to be closed with some time to act;<br></div><div>- Not automatically merging PRs, but warn the owner (and watchers) that the owner (if with commit rights) can merge the PR because of not having active reviews/comments for a while?</div><div><br></div><div>Can we also make possible to reactivate a PR once a new comment is done? I am thinking about the new contributers that see their PR being closed and have no way to Open it again. If they know no one, their work may be lost.</div><div><br></div><div>BTW, how much time of inactivity should trigger the bot? 2 weeks a warning, 3 weeks close? <br></div><div><br></div><div>Best regards,</div><div><br></div><div>Alex<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Paolo Cavallini <<a href="mailto:cavallini@faunalia.it">cavallini@faunalia.it</a>> escreveu no dia quarta, 16/01/2019 às 08:37:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
In case nobody objects, I'd suggest implementing this soon, during HF at<br>
latest, better before.<br>
All the best.<br>
<br>
On 16/01/19 08:44, Andreas Neumann wrote:<br>
> Hi Matteo,<br>
> <br>
> Your suggestion sounds good to me.<br>
> <br>
> Andreas<br>
> <br>
> Am 16.01.19 um 08:42 schrieb matteo:<br>
>> Hi Anita,<br>
>><br>
>>>      I am not very enthusiast of the automatic merging and closing of<br>
>>>      PRs. People have different velocities, and I don't think it's good<br>
>>>      to close a someones PR (that put some good work on it) just because<br>
>>>      no one with merge rights was able to review it. It is very<br>
>>> discouraging.<br>
>>><br>
>>><br>
>>> I think the suggestion was to automatically accept and merge them if<br>
>>> nobody with merge rights gets around to doing it.<br>
>> nope. What I'm suggesting is to automatic merge PR made by author having<br>
>> commit rights on the repo after a while and to close (or putting into<br>
>> stale) PR made by "external" people if there is not activity for a while.<br>
>><br>
>> This could speed up the cue of PR and prevent eventual breaking of docs.<br>
>><br>
>> Cheers<br>
>><br>
>> Matteo<br>
>> _______________________________________________<br>
>> QGIS-Developer mailing list<br>
>> <a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
>> List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
>> Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
> _______________________________________________<br>
> QGIS-Developer mailing list<br>
> <a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
> List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
> Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
<br>
-- <br>
Paolo Cavallini - <a href="http://www.faunalia.eu" rel="noreferrer" target="_blank">www.faunalia.eu</a><br>
<a href="http://QGIS.ORG" rel="noreferrer" target="_blank">QGIS.ORG</a> Chair:<br>
<a href="http://planet.qgis.org/planet/user/28/tag/qgis%20board/" rel="noreferrer" target="_blank">http://planet.qgis.org/planet/user/28/tag/qgis%20board/</a><br>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Alexandre Neto</div><div>---------------------</div><div>@AlexNetoGeo</div><div><a href="http://sigsemgrilhetas.wordpress.com">http://sigsemgrilhetas.wordpress.com</a></div><a href="http://gisunchained.wordpress.com">http://gisunchained.wordpress.com</a><br></div></div>