<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 28, 2017 at 4:04 PM, Yuya Nishihara <span dir="ltr"><<a href="mailto:yuya@tcha.org" target="_blank">yuya@tcha.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5">On Fri, 28 Apr 2017 10:30:00 -0700, Gregory Szorc wrote:<br>
> On Fri, Apr 28, 2017 at 6:04 AM, Yuya Nishihara <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>> wrote:<br>
> > On Thu, 27 Apr 2017 16:11:34 -0700, Gregory Szorc wrote:<br>
> > > This series is a "pick your poison" if you are a reviewer. I'm partial to<br>
> > > the more complex one adding an argument to @command. But it may not be<br>
> > > appropriate for stable. So a hacky fallback is provided as an<br>
> > alternative.<br>
> ><br>
> > I don't think either of them is good for stable. show is highly<br>
> > experimental<br>
> > extension, right?<br>
> ><br>
> > That said, the safest hack I can think of is adding null show command to<br>
> > the default command table, and wrap it by the show extension.<br>
> ><br>
><br>
> Oh, I forgot to mention in the commit message that this was the first thing<br>
> I did. It resulted in "show" appearing in help output, debugcomplete, etc.<br>
> Unless I was doing it wrong, I thought this was worse than hacking alias<br>
> resolution due to having user-visible side-effects.<br>
<br>
</div></div>Have you tried a command with no docstring (or "(EXPERIMENTAL)") ? I think<br>
the help issue can be addressed by that.<br></blockquote><div><br></div><div>No docstring displays "(no help text available)" in help.</div><div><br></div><div>Docstring of "(EXPERIMENTAL)" renders an entry in hgweb's help. (This smells like a bug since help's CLI output doesn't change.) show also appears in debugcomplete.<br></div><div><br></div><div>In both cases, we get a bunch of "extension 'show' overrides commands: show" warnings all over the place if we only do a simple @command in commands.py without hacking around the command table in show.py.<br></div></div><br></div></div>