From: | Dave Cramer <pg(at)fastcrypt(dot)com> |
---|---|
To: | Markus KARG <markus(at)headcrashing(dot)eu> |
Cc: | List <pgsql-jdbc(at)postgresql(dot)org> |
Subject: | Re: Git branching structure: ff or no-ff |
Date: | 2015-07-20 20:19:15 |
Message-ID: | CADK3HHJsLuWG_27Xa3V9LYf34qQNqOjS30ToXoSQWkHNiFgYTA@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc |
Well I can do it, it's not that difficult. But it does mean I can't do it
from the web.
Dave Cramer
dave.cramer(at)credativ(dot)ca
http://www.credativ.ca
On 20 July 2015 at 16:16, Markus KARG <markus(at)headcrashing(dot)eu> wrote:
> I checked the GitHub docs. Unfortunately --no-ff is the default when using
> the "Merge" button in the browser and it cannot be 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 the number of commits per PR to
> one per feature/bug covered by the PR.
>
> -Markus
>
>
> -----Original Message-----
> From: pgsql-jdbc-owner(at)postgresql(dot)org [mailto:
> pgsql-jdbc-owner(at)postgresql(dot)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(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>
>
>
> --
> Sent via pgsql-jdbc mailing list (pgsql-jdbc(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>
From | Date | Subject | |
---|---|---|---|
Next Message | Sehrope Sarkuni | 2015-07-20 22:19:44 | Re: Git branching structure: ff or no-ff |
Previous Message | Markus KARG | 2015-07-20 20:16:58 | Re: Git branching structure: ff or no-ff |