Re: [PATCH 1/2] git clone: check for non-empty directory
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Jul 2, 2020, 05:32 UTC
- Message-ID
- <CAPig+cQwJKSVMP8G7E62bESERYsAFTUN2PBX32YWMhECjU=qTQ@mail.gmail.com>
- In-Reply-To
- <CAPig+cT+cXuM3Asu6+Z25pGV2uRm6K1iAeYVhic9kqk1mS84-g@mail.gmail.com>
On Wed, Jul 1, 2020 at 12:00 PM Eric Sunshine <sunshine@sunshineco.com> wrote:
Show 8 quoted lines
> One other observation: To preserve bisectability (git-bisect) of > git.git itself, we want to ensure that the entire test suite still > passes at each point in a patch series. Making this change to the test > in its own patch introduces a failure into the test suite, which is > undesirable. One way to address this shortcoming would be to > temporarily change this test from 'test_must_succeed' to > 'test_must_fail', and then flip it back to 'test_must_succeed' in > patch 2/2. [...]
And another correction: I meant 'test_expect_success' and 'test_expect_failure', respectively.