<div dir="ltr"><div>Hi,</div><div><br></div><div>I am sorry for the late reply, I have just noticed this discussion.</div><div><br></div><div>I agree that we have a long time issue with Documentation. To me, it can only be solved with someone working on it "full-time", Which seems to be the case if Larissa starts working on it.</div><div><br></div><div>On the other hand, we have the review part, which is something that even fewer people participate (me included), and it takes forever...</div><div><br></div><div>I am not very enthusiast of the automatic merging and closing of PRs. People have different velocities, and I don't think it's good to close a someones PR (that put some good work on it) just because no one with merge rights was able to review it. It is very discouraging.</div><div><br></div><div>On the other hand, if the person has commit rights, then technically he can merge it himself , when he feels the work is ready (assuming that, like Matteo said it's not breaking anything), without the need for a bot. I still think that a review would be is required... Just like it happens in the Code side. Maybe Larissa can also review PRs?! That would speed thing up.<br></div><div><br></div><div>Now, I believe that we should not have opened tickets forever with hundreds of comments in it. <br></div><div><br></div><div>The geoserver team uses the following rule:</div><div><br></div><div>- At least one review is needed (beg it to a friend if needed)</div><div>- After addressing all the ISSUES found on the first review, the committer can merge. He should not wait for a second review to see if the new changes are ok, and then again and again...</div><div>- Any "good to have" observations/comments, should be moved to a new ticket for future improvements.</div><div><br></div><div>My 5 cents,</div><div><br></div><div>Alex Neto<br></div></div><br><div class="gmail_quote"><div dir="ltr">matteo <<a href="mailto:matteo.ghetta@gmail.com">matteo.ghetta@gmail.com</a>> escreveu no dia terça, 15/01/2019 às 12:06:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I found this:<br>
<br>
<a href="https://doc.mergify.io/index.html" rel="noreferrer" target="_blank">https://doc.mergify.io/index.html</a><br>
<br>
seems pretty straightforward to use (I'll make some test on test<br>
repositories).<br>
<br>
Somebody has already experiences on this side?<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></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>