<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 15, 2019 at 12:07 PM Huidae Cho <<a href="mailto:grass4u@gmail.com">grass4u@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi,</div><div><br></div><div>The oldest PR is 2 months old. Using git for collaboration is very different from svn, I believe. Unlike with svn, you have to "actively" merge these PRs into your branch for testing (e.g., features interesting to you?) and wait for an approval from reviewers (?). Why did it not happen to these old PRs? Now, a less bigger question is no one cares about them and who is supposed to be responsible for reviewing and approving them?<br></div></div></blockquote><div><br></div><div>Patches in svn tickets sometimes got old too. When no one cares (or has time), then it doesn't matter if we use svn or github. I would say PRs on github are easier to find and track than what we used before, although seem harder to test, but that's what we signed up for. </div><div>If you want someone's review, you can request it. If not, you can merge it yourself.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div><br></div><div>For example, <a href="https://github.com/OSGeo/grass/pull/52" target="_blank">https://github.com/OSGeo/grass/pull/52</a> fixes a very obvious formatting issue, which still needs to be approved? Tried to approve it myself, but self-nominating myself as a reviewer is pending... Hmm... Can I even merge it? The merge button seems to be activated. Confused...<br></div></div></blockquote><div><br></div><div>I actually asked in the related ticket for testing because I couldn't reproduce the problem (the formatting influences unicode/bytes). I didn't merge it because there is always a chance more changes might be needed and then you can update the PR and squash the commits and keep it nice and clean.</div><div><br></div><div>Anna</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div><br></div><div>Best,</div><div>Huidae<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 15, 2019 at 6:53 AM Markus Neteler <<a href="mailto:neteler@osgeo.org" target="_blank">neteler@osgeo.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Hi<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">Anna Petrášová <<a href="mailto:kratochanna@gmail.com" target="_blank">kratochanna@gmail.com</a>> schrieb am Mo., 15. Juli 2019, 05:57:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 12, 2019 at 12:50 PM Maris Nartiss <<a href="mailto:maris.gis@gmail.com" rel="noreferrer" target="_blank">maris.gis@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The biggest question is – do we need PR notifications here. I would<br>
vote for no. Better let's keep discussions in one place.<br></blockquote><div><br></div><div>I agree, and I get the notifications:</div><div><a href="https://help.github.com/en/articles/watching-and-unwatching-repositories" rel="noreferrer" target="_blank">https://help.github.com/en/articles/watching-and-unwatching-repositories</a></div></div></div></blockquote></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Then please all promise to subscribe to the repo notifications and look at them :)</div><div dir="auto"><br></div><div dir="auto">Thanks</div><div dir="auto">Markus</div><div dir="auto"><br></div><div dir="auto">PS: with "lost" I meant that nobody cares/d. Perhaps since many didn't subscribe to notifications yet...</div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div></div></div></blockquote></div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br></blockquote></div></div></blockquote></div></div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail-m_-7951457808210996397gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><span><div><div dir="ltr">Huidae Cho, Ph.D., GISP, PE (MD), CFM, M.ASCE<br>Open Source GIS Developer, GRASS GIS Development Team</div></div></span></div></div></div></div></div></div></div></div>
</blockquote></div></div>