Minor help tweaks to allow --help on even hidden commands to output something.

This won't allow those commands to show up in subcommand-help of their parents, however.

Author: baronfel

src/Cli/Microsoft.TemplateEngine.Cli/Help/HelpBuilder.cs

@@ -43,11 +43,6 @@ public virtual void Write(HelpContext context)
             throw new ArgumentNullException(nameof(context));
         }
 
-        if (context.Command.Hidden)
-        {
-            return;
-        }
-
         foreach (var writeSection in GetLayout(context))
         {
             if (writeSection(context))

src/Cli/dotnet/Parser.cs

@@ -307,17 +307,14 @@ public override void Write(HelpContext context)
         {
             var command = context.Command;
             var helpArgs = new string[] { "--help" };
+
+            // custom help overrides
             if (command.Equals(RootCommand))
             {
                 Console.Out.WriteLine(CliUsage.HelpText);
                 return;
             }
 
-            foreach (var option in command.Options)
-            {
-                option.EnsureHelpName();
-            }
-
             if (command.Equals(NuGetCommandParser.GetCommand()) || command.Parents.Any(parent => parent == NuGetCommandParser.GetCommand()))
             {
                 NuGetCommand.Run(context.ParseResult);
@@ -357,34 +354,38 @@ public override void Write(HelpContext context)
             {
                 new FsiForwardingApp(helpArgs).Execute();
             }
-            else
+
+            // argument/option cleanups specific to help
+            foreach (var option in command.Options)
             {
-                if (command.Name.Equals(ListReferenceCommandParser.GetCommand().Name))
-                {
-                    Command listCommand = command.Parents.Single() as Command;
+                option.EnsureHelpName();
+            }
 
-                    for (int i = 0; i < listCommand.Arguments.Count; i++)
+            if (command.Name.Equals(ListReferenceCommandParser.GetCommand().Name))
+            {
+                Command listCommand = command.Parents.Single() as Command;
+
+                for (int i = 0; i < listCommand.Arguments.Count; i++)
+                {
+                    if (listCommand.Arguments[i].Name == CliStrings.SolutionOrProjectArgumentName)
                     {
-                        if (listCommand.Arguments[i].Name == CliStrings.SolutionOrProjectArgumentName)
-                        {
-                            // Name is immutable now, so we create a new Argument with the right name..
-                            listCommand.Arguments[i] = ListCommandParser.CreateSlnOrProjectArgument(CliStrings.ProjectArgumentName, CliStrings.ProjectArgumentDescription);
-                        }
+                        // Name is immutable now, so we create a new Argument with the right name..
+                        listCommand.Arguments[i] = ListCommandParser.CreateSlnOrProjectArgument(CliStrings.ProjectArgumentName, CliStrings.ProjectArgumentDescription);
                     }
                 }
-                else if (command.Name.Equals(AddPackageCommandParser.GetCommand().Name) || command.Name.Equals(AddCommandParser.GetCommand().Name))
-                {
-                    // Don't show package completions in help
-                    PackageAddCommandParser.CmdPackageArgument.CompletionSources.Clear();
-                }
-                else if (command.Name.Equals(WorkloadSearchCommandParser.GetCommand().Name))
-                {
-                    // Set shorter description for displaying parent command help.
-                    WorkloadSearchVersionsCommandParser.GetCommand().Description = CliStrings.ShortWorkloadSearchVersionDescription;
-                }
-
-                base.Write(context);
             }
+            else if (command.Name.Equals(AddPackageCommandParser.GetCommand().Name) || command.Name.Equals(AddCommandParser.GetCommand().Name))
+            {
+                // Don't show package completions in help
+                PackageAddCommandParser.CmdPackageArgument.CompletionSources.Clear();
+            }
+            else if (command.Name.Equals(WorkloadSearchCommandParser.GetCommand().Name))
+            {
+                // Set shorter description for displaying parent command help.
+                WorkloadSearchVersionsCommandParser.GetCommand().Description = CliStrings.ShortWorkloadSearchVersionDescription;
+            }
+
+            base.Write(context);
         }
     }
 }