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
