jk/test-commit-bulk, was: What's cooking in git.git (Jul 2019, #05; Fri, 19)
- From
Jeff King <peff@peff.net>
- Date
- Jul 19, 2019, 21:54 UTC
- Message-ID
- <20190719215417.GA31841@sigill.intra.peff.net>
- In-Reply-To
- <xmqqef2lvkg5.fsf@gitster-ct.c.googlers.com>
On Fri, Jul 19, 2019 at 01:27:38PM -0700, Junio C Hamano wrote:
Show 15 quoted lines
> * jk/test-commit-bulk (2019-07-02) 7 commits > - SQUASH??? > - t6200: use test_commit_bulk > - t5703: use test_commit_bulk > - t5702: use test_commit_bulk > - t3311: use test_commit_bulk > - t5310: increase the number of bitmapped commits > - test-lib: introduce test_commit_bulk > > A test helper has been introduced to optimize preparation of test > repositories with many simple commits, and a handful of test > scripts have been updated to use it. > > Needs a bit more polishing. > I think I spotted one unused command, which we may want to remove, though.
I think these comments are out-of-date. The unused command was fixed up in v2 of the first patch, and then I did a little extra polishing in v3:
https://public-inbox.org/git/20190702051649.GA16344@sigill.intra.peff.net/
Together with your SQUASH fixup, I think that covers all comments. For convenience, here's v3 again as a whole series which can replace what you have already (including the squash).
[1/6]: test-lib: introduce test_commit_bulk [2/6]: t5310: increase the number of bitmapped commits [3/6]: t3311: use test_commit_bulk [4/6]: t5702: use test_commit_bulk [5/6]: t5703: use test_commit_bulk [6/6]: t6200: use test_commit_bulk
t/t3311-notes-merge-fanout.sh | 10 +-- t/t5310-pack-bitmaps.sh | 15 +--- t/t5702-protocol-v2.sh | 10 +-- t/t5703-upload-pack-ref-in-want.sh | 4 +- t/t6200-fmt-merge-msg.sh | 7 +- t/test-lib-functions.sh | 123 +++++++++++++++++++++++++++++ 6 files changed, 136 insertions(+), 33 deletions(-)
-Peff