<div dir="ltr"><div>I tend to agree with Yves on this. Long review processes will just push contributors away (if we ever attract them to contribute anyway).<br></div><div><br></div><div>On the limit, we must make sure each PR is accurate regarding its content, it is well placed, and don't break the build. Enhancing wording and grammar, fixing small typos, and improve Sphinx formatting could be done afterwords, just before the release.<br></div><div><br></div><div>I think that would be the perfect way to spend Documentation funds, hiring an English native tech writer (no need to know much of QGIS) to review the documentation and fix wording, grammar, spelling, formatting and so on.</div><div><br></div><div>I am not saying we should merge PRs badly written and full of typos, but we should not nitpick too much in search for perfection. Better to have some low quality english documentation about a complex QGIS feature, than having none.</div><div><br></div><div>I also agree that we may have to fix an already released version of the documentation, and not wait months to release a perfect one. If QGIS LTR receives patches to fix issues, why can't documentation do the same?<br></div><div><br></div><div>Regarding editing someone's else PR, I agree with Harrissou that, if a valid PR is left untouched for too long after a review, instead of abandoning it, we (core contributors) could/should take it as our own, fix any remarkable issues and merge the content. That way, the work is not lost.</div><div><br></div><div> I am also in favor of allowing people to edit my typos and bad sphinx formatting instead of asking me to do it, wasting twice the time. I am not so keen to receive changes to wording and sentences without some discussion. But I guess that is something each contributor has to decide together with the reviewer.</div><div><br></div><div>Best regards,</div><div><br></div><div>Alex</div><div><br></div><div>PS: This being said, I will try to do some reviews (mainly to the content) in the next days/weeks. Let's reduce the number of opened PRs! It make us look bad! :-P</div><div><br></div><div><br></div><div><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 quarta, 26/09/2018 às 08:35:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
Le 20/09/2018 à 10:25, DelazJ a écrit :<br>
> About some topics discussed in this thread, I'm less enthusiastic with<br>
> what I seem to understand.<br>
> I personally really prefer to merge a clean PR  without issues (and<br>
> sorry if ever I bothered some with all my (nit-?)picking), than<br>
> letting issues I'm aware of being merged and hope that someone else<br>
> (who?) will later treat them (when?). And if those issues are not<br>
> fixed before release, what about translators workload? Do we ask them<br>
> to translate the doc as is and if ever the issue is fixed, to<br>
> translate those strings again? Or do we consider that once a doc is<br>
> released, we do not touch it again (in which case what about the<br>
> issues we let pass in the PR)?<br>
> As said, maybe I misunderstand the topic.<br>
<br>
The topic seams clear: how to speed up (and, so, make PR easier). If I<br>
understood you well, you propose to not change anything.<br>
<br>
You said:<br>
<br>
> And if those issues are not fixed before release<br>
<br>
This is based in case of a release, what if we can't release the<br>
documentation before, let's say, one year? Why do we need to think to<br>
translator if we can't make a new release?<br>
<br>
Second, if we can make minor release of QGIs, why we can't do it for<br>
Documentation? Why can't we based the workflow on "release early,<br>
release often" princips? Translator will update a few string after each<br>
release, and so? Transifex allow to find previous translation from<br>
similar string, we won't change all the documentation, just some string.<br>
That's not seem a big work.<br>
<br>
I do still think that higher(st) quality is counter productive and I<br>
think (but not hope) that in few months we won't increase the number of<br>
contributor (rather decrease it).<br>
<br>
I am open to new propositions but not to let this process as it<br>
currently is.<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>