Re: [PATCH v3 1/3] ref-filter: add worktreepath atom

2018-12-20 Thread Jeff King
On Wed, Dec 19, 2018 at 11:09:59PM -0800, Nickolai Belakovski wrote: > > Also, why are we replacing it with a single space? Wouldn't the empty > > string be more customary (and work with the other "if empty, then do > > this" formatting options)? > > I was just following what was done for HEAD, b

Re: [PATCH v3 1/3] ref-filter: add worktreepath atom

2018-12-19 Thread Nickolai Belakovski
On Tue, Dec 18, 2018 at 9:22 AM Jeff King wrote: > > On Sun, Dec 16, 2018 at 01:57:57PM -0800, nbelakov...@gmail.com wrote: > > > From: Nickolai Belakovski > > > > Add an atom proving the path of the linked worktree where this ref is > > checked out, if it is checked out in any linked worktrees,

Re: [PATCH v3 1/3] ref-filter: add worktreepath atom

2018-12-18 Thread Jeff King
On Sun, Dec 16, 2018 at 01:57:57PM -0800, nbelakov...@gmail.com wrote: > From: Nickolai Belakovski > > Add an atom proving the path of the linked worktree where this ref is > checked out, if it is checked out in any linked worktrees, and empty > string otherwise. I stumbled over the word "provi