Re: [PATCH 1/2] add basic branch display tests
- From
Jeff King <peff@peff.net>
- Date
- Feb 19, 2009, 03:53 UTC
- Message-ID
- <20090219035332.GC19862@coredump.intra.peff.net>
- In-Reply-To
- <76718490902181951p54dc222ai5ac6397f3471fc44@mail.gmail.com>
On Wed, Feb 18, 2009 at 10:51:27PM -0500, Jay Soffian wrote:
Show 10 quoted lines
> On Wed, Feb 18, 2009 at 10:45 PM, Jeff King <peff@peff.net> wrote: > > But here we stick the "remotes/" head on, since we are showing both > > types. But the right hand side of the symref doesn't get the same > > treatment. > > > > I don't think it's a big deal, but I wasn't sure if it was intentional, > > It was intentional on my part. I thought appending it a second time > was redundant, and the remotes/ prefix on the LHS is just their to > distinguish remote branches from local.
OK. The more I think about it, the more I think what is currently there is best.
> Thanks for the tests. If I had added them myself I wouldn't have you > publicly questioning my intent. ;-)
You're welcome. :)
-Peff