Re: Git branching structure: ff or no-ff

We noticed that this happens a lot on github in other projects, too, while it does not happen on our local company
repos.Possibly a bug of the github platform itself? 

About the "fix here, fix there": I think it is up to the project lead to reject non-collapsed PRs. I'd vote for such a
rule,as a shared master branch should only contain single commits per bug/feature. 

-Markus

-----Original Message-----
From: pgsql-jdbc-owner@postgresql.org [mailto:pgsql-jdbc-owner@postgresql.org] On Behalf Of Vladimir Sitnikov
Sent: Sonntag, 19. Juli 2015 04:06
To: List
Subject: [JDBC] Git branching structure: ff or no-ff

Hi,

Any clue why --no-ff merges are used a lot in pgjdbc development?
Any clue why "fix here, fix there" commits are not squashed before integration?

From my point of view, it makes history browsing hard:
1) Just look at https://github.com/pgjdbc/pgjdbc/commits/master
Commits on Jul 9, 2015
Merge pull request #333 from zapov/master … Merge pull request #343 from phillipross/master … Merge pull request #351
fromheadcrashing/#328 … Merge pull request #349 from ekoontz/jsonb-support … ... 
Does that make much sense?

2) Even a single commit becomes two commits, so it clutters change log

3) It looks odd to have all those "fix here, fix there" commits in the final history:
https://github.com/pgjdbc/pgjdbc/pull/343/commits


What everyone thinks on that?

http://endoflineblog.com/gitflow-considered-harmful
http://endoflineblog.com/follow-up-to-gitflow-considered-harmful
https://news.ycombinator.com/item?id=9745966

--
Regards,
Vladimir Sitnikov


--
Sent via pgsql-jdbc mailing list (pgsql-jdbc@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-jdbc