<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 21, 2019 at 7:48 AM Markus Neteler <<a href="mailto:neteler@osgeo.org">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">On Tue, May 21, 2019 at 11:23 AM Bas Couwenberg <<a href="mailto:sebastic@xs4all.nl" target="_blank">sebastic@xs4all.nl</a>> wrote:<br>
> On 2019-05-21 11:22, Markus Neteler wrote:<br>
> > How to revert that? Of course someone had to run into this at some<br>
> > point, so seems I "volunteered" to be the first :((<br>
><br>
> rebase and force push.<br>
<br>
Thanks - we investigated further with two local git gurus, it seems<br>
that nothing happened and things are indeed ok.<br>
</blockquote><br>Well, there is "Merge branch 'master' of github.com:OSGeo/grass" commit which happens when your local repo was not up to date, you did a commit, and you did git pull for Git to allow you to do git push.<br><div> </div><div>While commits like "Merge pull request #2 from pmav99/t3844" seems equally messy, they 1) include only the relevant changes and 2) they tell us when the changes were merged into master.  "Merge branch 'master' of github.com:OSGeo/grass" commit contains all sorts of changes and what it tells us is when you updated your local repository clone which is comparatively less useful information.</div><div><br></div><div>Commits like "Merge branch 'master' of github.com:OSGeo/grass" are unavoidable with pushing directly to master unless you use rebase when catching up with upstream changes. Unless you are careful and do rebase every time (or set your Git to do that), this will happen every time you are out of sync with upstream changes, so potentially once for every commit. Thus, we should be using forks and PRs (every time) until we decide otherwise.</div><div><br></div><div>If something happened or not depends on if we want a commit like  "Merge branch 'master' of github.com:OSGeo/grass" in the history. In our lab repos, we have plenty of these commits because in most cases we don't care about clean commit history and often only one person works on the repo content at one time (so no sync is needed). Anyway, this commit does *not* show in the individual file history or blame on GitHub. I don't know about git bisect, but since you would operate over whole repository, I assume it would show up which might be quite confusing if it is indeed the case.<br></div><div><br></div><div>Vaclav<br></div><div><br></div><div><a href="https://github.com/OSGeo/grass/commits/master/README.md">https://github.com/OSGeo/grass/commits/master/README.md</a></div><div><a href="https://github.com/OSGeo/grass/blame/master/README.md">https://github.com/OSGeo/grass/blame/master/README.md</a></div></div></div>