Add missing nameMapper flag to .runOrExit (#155)

Also fix `runOrEither`s `sort` flag to make it default to `true` as the
doxs suggest it should

Author: lihaoyi

mainargs/src/Parser.scala

@@ -54,7 +54,9 @@ class ParserForMethods[B](val mains: MethodMains[B]) {
       docsOnNewLine: Boolean = false,
       autoPrintHelpAndExit: Option[(Int, PrintStream)] = Some((0, System.out)),
       customNames: Map[String, String] = Map(),
-      customDocs: Map[String, String] = Map()
+      customDocs: Map[String, String] = Map(),
+      sorted: Boolean = true,
+      nameMapper: String => Option[String] = Util.kebabCaseNameMapper
   ): Any = {
     runEither(
       args,
@@ -65,7 +67,9 @@ class ParserForMethods[B](val mains: MethodMains[B]) {
       docsOnNewLine,
       autoPrintHelpAndExit,
       customNames,
-      customDocs
+      customDocs,
+      sorted,
+      nameMapper
     ) match {
       case Left(msg) =>
         stderr.println(msg)
@@ -74,6 +78,32 @@ class ParserForMethods[B](val mains: MethodMains[B]) {
     }
   }
 
+  def runOrExit(
+      args: Seq[String],
+      allowPositional: Boolean,
+      allowRepeats: Boolean,
+      stderr: PrintStream,
+      totalWidth: Int,
+      printHelpOnExit: Boolean,
+      docsOnNewLine: Boolean,
+      autoPrintHelpAndExit: Option[(Int, PrintStream)],
+      customNames: Map[String, String],
+      customDocs: Map[String, String]
+  ): Any = {
+    runOrExit(
+      args,
+      allowPositional,
+      allowRepeats,
+      stderr,
+      totalWidth,
+      printHelpOnExit,
+      docsOnNewLine,
+      autoPrintHelpAndExit,
+      customNames,
+      customDocs,
+    )
+  }
+
   def runOrThrow(
       args: Seq[String],
       allowPositional: Boolean,
@@ -135,7 +165,7 @@ class ParserForMethods[B](val mains: MethodMains[B]) {
       autoPrintHelpAndExit: Option[(Int, PrintStream)] = Some((0, System.out)),
       customNames: Map[String, String] = Map(),
       customDocs: Map[String, String] = Map(),
-      sorted: Boolean = false,
+      sorted: Boolean = true,
       nameMapper: String => Option[String] = Util.kebabCaseNameMapper
   ): Either[String, Any] = {
     if (autoPrintHelpAndExit.nonEmpty && args.take(1) == Seq("--help")) {

mainargs/test/src/EqualsSyntaxTests.scala

@@ -42,9 +42,9 @@ object EqualsSyntaxTests extends TestSuite {
       ParserForMethods(Main).runEither(Array("--foo=bar", "--bool=true")) ==>
          Left("""Unknown argument: "--bool=true"
                 |Expected Signature: run
+                |  --bool          Example flag
                 |  -f --foo <str>  String to print repeatedly
                 |  --my-num <int>  How many times to print string
-                |  --bool          Example flag
                 |
                 |""".stripMargin)
     }