Re: [PATCH v5 2/9] builtin/refs: support multiple worktrees check for refs.
On Mon, Oct 07, 2024 at 11:16:19AM +0200, Patrick Steinhardt wrote:
[snip]
Show 19 quoted lines
> > However, the refname would be printed to "refs/worktree/test". It will
> > make the user confused which "refs/worktree/test" is checked. So, we
> > should print this information like:
> >
> > Checking references consistency in .git
> > ...
> > checking references consistency in .git/worktrees/A
> > ...
> > checking references consistency in .git/worktrees/B
> >
> > However, when writing this, I feel a ".git" is a bad usage. It will make
> > the user think it will check everything here. This should be improved in
> > the next version.
>
> But wouldn't it be the better solution if we printed the fully-qualified
> reference name "worktrees/worktree/refs/worktree/test" instead? That
> would remove the need to say which directory we're currently verifying
> in the first place.
>
Good idea. I will use this way in the next version.
> Patrick