Fix short for flag not being applied (#4)

indoorvivants · Aug 6, 2024 · 6b7b583 · 6b7b583
1 parent 98ac852
commit 6b7b583
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/library.test.scala b/library.test.scala
@@ -68,10 +68,12 @@ class Tests extends FunSuite:
     assertArgs[Cmd](Cmd(Some("shroom"), true))("--yepp", "shroom", "--flag2")
 
   test("argument hints: short"):
-    case class Cmd(@arg(_.Short("y")) location: Option[String])
-        derives CommandApplication
+    case class Cmd(
+        @arg(_.Short("y")) location: Option[String],
+        @arg(_.Short("X")) flag: Boolean
+    ) derives CommandApplication
 
-    assertArgs[Cmd](Cmd(Some("shroom")))("-y", "shroom")
+    assertArgs[Cmd](Cmd(Some("shroom"), true))("-y", "shroom", "-X")
 
   test("argument hints: flag default"):
     case class Cmd(@arg(_.FlagDefault(true)) isLit: Boolean)

diff --git a/macros.scala b/macros.scala
@@ -186,15 +186,17 @@ private[decline_derive] object Macros:
             case None =>
               Opts
                 .flag(
-                  ${ hints.name }.getOrElse($nm),
-                  ${ hints.help }.getOrElse("")
+                  long = ${ hints.name }.getOrElse($nm),
+                  help = ${ hints.help }.getOrElse(""),
+                  short = ${ hints.short }.getOrElse("")
                 )
                 .orFalse
             case Some(value) =>
               Opts
                 .flag(
-                  ${ hints.name }.getOrElse($nm),
-                  ${ hints.help }.getOrElse("")
+                  long = ${ hints.name }.getOrElse($nm),
+                  help = ${ hints.help }.getOrElse(""),
+                  short = ${ hints.short }.getOrElse("")
                 )
                 .orTrue
         }