Re: [PATCH 3/5] pull: handle conflicting rebase/merge options via last option wins
- From
Elijah Newren <newren@gmail.com>
- Date
- Jul 15, 2021, 17:13 UTC
- Message-ID
- <CABPp-BENqG5Mr4uPzbNQOxaj--f+mMe=fbeC1UTLA_zwqX34ew@mail.gmail.com>
- In-Reply-To
- <CAPig+cRZ7gFACO9eNtGmXjt_SG2oviXR7JjuW2w0wZeV7pMjkQ@mail.gmail.com>
On Wed, Jul 14, 2021 at 10:00 PM Eric Sunshine <sunshine@sunshineco.com> wrote:
Show 18 quoted lines
> > On Wed, Jul 14, 2021 at 10:41 PM Elijah Newren via GitGitGadget > <gitgitgadget@gmail.com> wrote: > > The --rebase[=...] flags and the various ff flags are incompatible, > > except that --no-rebase (or --rebase=false) work with any of the ff > > flags, and --ff works with any of the rebase flags. > > [...] > > Signed-off-by: Elijah Newren <newren@gmail.com> > > --- > > diff --git a/t/t7601-merge-pull-config.sh b/t/t7601-merge-pull-config.sh > > +test_expect_success 'report conflicting configuration' ' > > + git reset --hard c2 && > > + test_must_fail git -c pull.ff=false -c pull.rebase=true pull . c1 2>err && > > + test_i18ngrep "pull.rebase and pull.ff are incompatible; please unset one" err > > + > > +' > > nit: unnecessary and inconsistent blank line at end of test
Good catch, will fix (assuming the relevant folks agree with my general --ff-only & --no-ff vs. --rebase handling approach).