On Sun, Sep 19, 2021 at 10:47:19AM +0200, Ævar Arnfjörð Bjarmason wrote: > There's no corresponding config for "command_requires_full_index", but > let's change it too for consistency and to prevent future bugs > creeping in due to one of these being "unsigned". This was the only thing that I thought of while reading through the diff, so I appreciate that you explicitly called out why command_requires_full_index was changed in this patch. This looks good to me. Thanks, Taylor