<div dir="ltr">Hi,<br><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 8, 2018 at 11:56 AM, Ari Jolma <span dir="ltr"><<a href="mailto:ari.jolma@gmail.com" target="_blank">ari.jolma@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
I made a small PR yesterday - and accidentally added another small fix to it today. The PR is at <a href="https://github.com/qgis/QGIS/pull/6556" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/p<wbr>ull/6556</a><br>
<br>
The first change in the PR is a fix to translation (a string is changed before it is sent to tr) and the second change fixes a missing default to gdaldem color-relief processing (I suspect many more fixes like this needed to other gdal processing tools, see my old PR 4220)<br></blockquote><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I'm not very familiar with PRs, especially in a large project like QGIS. It seems to me that I should specifically ask the original authors of those sources I'm fixing to review the PR? And should I assume they will merge the PRs? The travis CI build seems to have failed checking my PR - but most likely the reason is not my PR but something else - how to check?<br>
<br>
Lastly, these changes are very small, is PR the correct way to submit them?<br></blockquote><div><br></div><div>yes, you did it right.<br clear="all"></div></div><br></div><div class="gmail_extra">You can politely ask a developer to review your PR in a comment in your PR but there is no guarantee that the developer will have time to review it.<br><br></div><div class="gmail_extra">Most developers go through the PR queue from time to time, but not every developer is expert in all areas of the code.<br><br></div><div class="gmail_extra">You can check the failures on travis: <a href="https://travis-ci.org/qgis/QGIS/builds/350755583?utm_source=github_status&utm_medium=notification">https://travis-ci.org/qgis/QGIS/builds/350755583?utm_source=github_status&utm_medium=notification</a><br><br></div><div class="gmail_extra">Keep in mind that sometimes Travis fails for obscure reasons that are not related to the PR, if this is the case closing and re-opening the PR will trigger Travis again and if it was a temporary problem the tests will eventually pass.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Btw, thanks for contributing to QGIS with your patches!<br><br></div><div class="gmail_extra">-- <br><div class="gmail_signature">Alessandro Pasotti<br>w3:   <a href="http://www.itopen.it" target="_blank">www.itopen.it</a></div>
</div></div>