tristan/cligen
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Added differnt check for default. Set it to empty string as a way of determining if a default was set. Ignoring empty strings

Browse Source
This commit is contained in:
Tristan Ancelet
2026-08-14 15:01:04 -05:00
parent d1746602b1
commit 09389cb5e9
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/cligen/flag.cr
+2 -2
View File
@@ -6,7 +6,7 @@ module CliGen
type : String, type : String,
array : Bool, array : Bool,
format : String?, format : String?,
default : String?, default : String,
options : Array(String)? options : Array(String)?
abstract class BaseFlag abstract class BaseFlag
@@ -81,7 +81,7 @@ module CliGen
default = @default.not_nil!.to_s default = @default.not_nil!.to_s
{% end %} {% end %}
else else
default = nil default = ""
end end
meta = FlagMeta.new( meta = FlagMeta.new(
type: {{T.stringify}}, type: {{T.stringify}},
src/cligen/template/cmd_help.ecr
+1 -1
View File
@@ -14,7 +14,7 @@ Flags:
<%- else -%> <%- else -%>
<%- flags = [flag.short,flag.long.strip] -%> <%- flags = [flag.short,flag.long.strip] -%>
<%- end -%> <%- end -%>
<%= "%-#{len}s %s" % [flags.join(","), flag.description.strip] %><%= flag.meta.options.nil? ? "" : " (valid: #{flag.meta.options.not_nil!.join(", ")})" %><%= flag.meta.default.nil? ? "" : " (default: #{flag.meta.default.not_nil!})" %> <%= "%-#{len}s %s" % [flags.join(","), flag.description.strip] %><%= flag.meta.options.nil? ? "" : " (valid: #{flag.meta.options.not_nil!.join(", ")})" %><%= flag.meta.default.empty? ? "" : " (default: #{flag.meta.default.not_nil!})" %>
<%- if verbose? -%> <%- if verbose? -%>
<%= "%-#{len}s %s" % ["", "Type: #{flag.meta.type}"] %> <%= "%-#{len}s %s" % ["", "Type: #{flag.meta.type}"] %>
<%= "%-#{len}s %s" % ["", "ENV VAR: #{flag.env_var}"] %> <%= "%-#{len}s %s" % ["", "ENV VAR: #{flag.env_var}"] %>