From 5eae72b4c184c3b1f83776e3986a5283cc84f0e7 Mon Sep 17 00:00:00 2001
From: Lyssieth <lys@the.cutie.zone>
Date: Thu, 28 Nov 2024 18:12:06 +0200
Subject: [PATCH] Minor improvements

Let's not mark toggles as required... they shouldn't be.
---
 src/args/help.zig | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/args/help.zig b/src/args/help.zig
index 41befde..a58e9d3 100644
--- a/src/args/help.zig
+++ b/src/args/help.zig
@@ -151,13 +151,15 @@ pub fn printHelp(comptime T: type, comptime name: []const u8, writer: std.io.Any
                     try writer.print(")", .{});
                 }
 
-                try writer.print(": {s}", .{valueType});
+                if (flag.takesValue) {
+                    try writer.print(": {s}", .{valueType});
 
-                if (flag.typeHint) |typeHint| {
-                    try writer.print(" ({s})", .{typeHint});
+                    if (flag.typeHint) |typeHint| {
+                        try writer.print(" ({s})", .{typeHint});
+                    }
                 }
 
-                if (!isOptional) {
+                if (!isOptional and !flag.toggle) {
                     try writer.print(" <required>", .{});
                 }
 
@@ -199,7 +201,7 @@ test "basic help" {
                 .Flag = .{
                     .name = "verbose",
                     .short = "v",
-                    .takesValue = false,
+
                     .toggle = true,
                 },
             },
@@ -221,7 +223,7 @@ test "basic help" {
         \\Usage: demo [flags] <positional> [...]
         \\Legend: <required> [optional]
         \\
-        \\* --verbose (-v): bool <required>
+        \\* --verbose (-v)
         \\* <positional>: string
         \\
     , buf.items);