Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Seperator does not appear in help text when diplaying default values #490

Open
sorvis opened this issue Aug 1, 2019 · 1 comment · May be fixed by #895
Open

Seperator does not appear in help text when diplaying default values #490

sorvis opened this issue Aug 1, 2019 · 1 comment · May be fixed by #895
Labels
enhancement

Comments

@sorvis
Copy link

sorvis commented Aug 1, 2019

I have the following property in the options:

[Option("branch", Required = false, Separator = ',', HelpText = "Branches to include.",
        Default = new[] { "master", "breakfix" })]
public IEnumerable<string> IncludeBranches { get; set; }

The separator is set to ',' then a list of string is given for the default.

The help output is:

--branch                (Default: master breakfix) Branches to include.

This would be miss-leading to the user since it does not include the comma. Would it instead make sense to do something like this?

--branch                (Default: master,breakfix) Branches to include.
@moh-hassan
Copy link
Collaborator

PR is welcome.

gfs added a commit to gfs/commandline that referenced this issue Jul 27, 2023
@gfs
Display appropriate separator when writing default enumerable values …
89a8e78 
…in HelpText

Fix commandlineparser#490
This was referenced Jul 27, 2023
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #490: Display appropriate separator when writing default enumerable values gfs/commandline
2 participants
@sorvis @moh-hassan