On Fri, Apr 28, 2017 at 6:04 AM, Yuya Nishihara <y...@tcha.org> wrote:
> On Thu, 27 Apr 2017 16:11:34 -0700, Gregory Szorc wrote: > > This series is a "pick your poison" if you are a reviewer. I'm partial to > > the more complex one adding an argument to @command. But it may not be > > appropriate for stable. So a hacky fallback is provided as an > alternative. > > I don't think either of them is good for stable. show is highly > experimental > extension, right? > > That said, the safest hack I can think of is adding null show command to > the default command table, and wrap it by the show extension. > Oh, I forgot to mention in the commit message that this was the first thing I did. It resulted in "show" appearing in help output, debugcomplete, etc. Unless I was doing it wrong, I thought this was worse than hacking alias resolution due to having user-visible side-effects.
_______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel