On Mon, Sep 13, 2021 at 11:12:21AM -0700, Glen Choo wrote:
builtin/gc.c has two ways of checking of multi-pack-index is enabled:
- git_config_get_bool() in incremental_repack_auto_condition()
- the_repository->settings.core_multi_pack_index in
maintenance_task_incremental_repack()
Is this hinting at why there are no new tests added here? If so, it may
need to be explained more clearly, since I was a tad confused after
reading it. But if not, this patch message deserves an extra sentence or
two saying why tests aren't necessary.
Or if none of that is the case, and tests *are* necessary, then they
should be added ;).
Thanks,
Taylor