On Wed, Jun 21, 2023 at 07:35:10PM +0000, John Cai via GitGitGadget wrote: > From: John Cai <johncai86@xxxxxxxxx> > > The ref_exclusions API provides the ability to check if certain refs are > to be excluded. We can easily extend this API to check if certain refs > are included, which [1] considered when teaching git-pack-refs the > ability to specify not only refs to exclude but ones to include. > > A subsequent commit will actuall extend the API to add the ability to > keep track of ref inclusions. As a preparatory patch, rename > ref_exclusions to ref_visibility. Skimming through this patch, it looks like a straight-forward rename that doesn't change any functionality. I think other readers may benefit from a note that says something to that effect. > --- > builtin/pack-refs.c | 6 ++-- > builtin/rev-parse.c | 18 +++++----- > refs.h | 2 +- > refs/files-backend.c | 2 +- > revision.c | 72 +++++++++++++++++++-------------------- > revision.h | 18 +++++----- > t/helper/test-ref-store.c | 4 +-- > 7 files changed, 61 insertions(+), 61 deletions(-) Obviously all of this looks OK. Thanks, Taylor