Re: [PATCH v2 02/18] Add a new builtin: branch-diff

From: Jeff King <peff@peff.net>
Date: May 5, 2018, 18:26 UTC
Message-ID: <20180505182631.GC17700@sigill.intra.peff.net>
In-Reply-To: <a1ea0320b64527ee6ce9856dcf359513d13052b7.1525448066.git.johannes.schindelin@gmx.de>

On Fri, May 04, 2018 at 05:34:32PM +0200, Johannes Schindelin wrote:

> This builtin does not do a whole lot so far, apart from showing a usage
> that is oddly similar to that of `git tbdiff`. And for a good reason:
> the next commits will turn `branch-diff` into a full-blown replacement
> for `tbdiff`.

One minor point about the name: will it become annoying as a tab completion conflict with git-branch?

It feels really petty complaining about the name, but I just want to raise the point, since it will never be easier to change than right now.

(And no, I don't really have another name in mind; I'm just wondering if "subset" names like this might be a mild annoyance in the long run).

-Peff

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.