Re: Git branching structure: ff or no-ff - Mailing list pgsql-jdbc

From Markus KARG
Subject Re: Git branching structure: ff or no-ff
Date
Msg-id 004c01d0c329$08376d30$18a64790$@eu
Whole thread Raw
In response to Re: Git branching structure: ff or no-ff  (Vladimir Sitnikov <sitnikov.vladimir@gmail.com>)
Responses Re: Git branching structure: ff or no-ff
List pgsql-jdbc
I checked the GitHub docs. Unfortunately --no-ff is the default when using the "Merge" button in the browser and it
cannotbe changed! To have use fast-forward, Dave must always pull the change locally and merge it by hand. 

So for simplicitly we should live with --no-ff to make Dave's life easier, but I think it should be common to reduce
thenumber of commits per PR to one per feature/bug covered by the PR. 

-Markus


-----Original Message-----
From: pgsql-jdbc-owner@postgresql.org [mailto:pgsql-jdbc-owner@postgresql.org] On Behalf Of Vladimir Sitnikov
Sent: Montag, 20. Juli 2015 20:35
To: Dave Cramer
Cc: List
Subject: Re: [JDBC] Git branching structure: ff or no-ff

I think we'd better keep existing commits as is otherwise everybody
will have problems with on-going pull requests.

For new commits I suggest we use certain level of rebasing.
I suggest rebasing features on top of master before integration, so
the history is linear.

What do you think?

For instance: commit cddcd185  feat: Improved composite/array type support...
git checkout -b tmp cddcd185 # git fetch origin
pull/ID/head:BRANCHNAME && git checkout head:BRANCHNAME can be used
here to fetch a pull request from github
git rebase 008f6e8b2f8d # use rebase -i to squash commits as required.
008f6e8b2f8d was origin/master before cddcd185 merged in
git checkout master
git merge --ff-only tmp
git branch -D tmp # remove branch

Vladimir


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



pgsql-jdbc by date:

Previous
From: Vladimir Sitnikov
Date:
Subject: Re: Git branching structure: ff or no-ff
Next
From: Dave Cramer
Date:
Subject: Re: Git branching structure: ff or no-ff