<div dir="ltr"><div>Hi Yves, thanks for raising this matter.</div><div><br></div><div>+1 for making it easier to accept and merge contributions.<br></div><div><br></div><div>I would just swap the "There are typos" with the "location of the contributions" position. Mainly because the typos can be fixed later (unless there are major English problem that make the text difficult to understand), where contribution location can mess things up badly, and might be more difficult to untangle later. <br></div><div><br></div><div>Maybe, we should open dedicated tickets for reviewing and "sphinxifying" content for non important and no blocker merged PRs (or remove some tags and add some others (Screenshots, Needs English Review, Sphinx)).<br></div><div><br></div><div> Maybe we could use QGIS funds to hire someone that could tackle those "after content writing" tickets, which may be much easier than finding candidates that are both QGIS (advanced) users, good writing technical English, and familiar with sphinx syntax.</div><div><br></div><div>Thanks,</div><div><br></div><div>Alexandre Neto<br></div></div><br><div class="gmail_quote"><div dir="ltr">Yves Jacolin <<a href="mailto:yjacolin@free.fr">yjacolin@free.fr</a>> escreveu no dia terça, 18/09/2018 às 16:45:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello community and documentation contributors,<br>
<br>
I would like to raise a discussion about the merge process in the QGIS<br>
documentation.<br>
<br>
My opinion is that the documentation process should be made easier in<br>
order to increase the dynamic and the speed of feature documentation. I<br>
do think we try to do our best to do some doc review, but probably not<br>
enough and not fast enough. We should improve this and be faster in<br>
order to be closer to the QGIS feature of the current public release,<br>
then look at how to improve documentation later. If features are well<br>
documented, we can probably pay for someone to proof read and fix<br>
english issue.<br>
<br>
In this purpose I would like to propose some keys to help merging PR and<br>
I would like to get your opinion.<br>
<br>
I would like to create three "levels" (a kind of guidelines) to<br>
accept/refuse a PR, in a similar way as QGIS issues:<br>
<br>
# Blocker (="need change" in review process)<br>
<br>
Impossible to merge the PR because:<br>
<br>
* Travis failed<br>
* There is typos<br>
* Write does not use any rst tags (:guilabel:, code-block, index, title,<br>
etc.)<br>
<br>
# No blocker but important (="comment" in review process)<br>
<br>
PR can be merged but contributors can improve it before merging because:<br>
<br>
* There is some missing rst tag<br>
* Some sentences are not well written (I mean most of doc contributor<br>
are not native english writer and so we are nos using the best english,<br>
this email is a good illustration)<br>
<br>
# Not important, just feedback (="comment" in review process)<br>
<br>
PR can be merged, this is mainly subjective or small changes as:<br>
<br>
* location of the contribution<br>
* need screenshot (or update)<br>
* fix a problem which has not been created by the PR<br>
* Not enough description<br>
<br>
Other contribution can create issue to not forget some of this changes<br>
proposals.<br>
<br>
We already agreed on the following, but better to tell it again: small<br>
fix (add rst tag, fix typos) could be merged as soon as travis is a<br>
success! So please, focus in small fix outside bigger PR :)<br>
<br>
I am looking forward to read your opinion :)<br>
<br>
Thanks to all for your great contributions!<br>
<br>
Y.<br>
<br>
-- <br>
<a href="http://yjacolin.gloobe.org" rel="noreferrer" target="_blank">http://yjacolin.gloobe.org</a><br>
<br>
<br>
_______________________________________________<br>
Qgis-community-team mailing list for organizing community resources such as documentation, translation etc..<br>
<a href="mailto:Qgis-community-team@lists.osgeo.org" target="_blank">Qgis-community-team@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/qgis-community-team" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-community-team</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>