Skip to content

ArgParser: enable default command and help message update #4343

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
Merged
dragon512 merged 1 commit into from
Oct 9, 2018
Merged

ArgParser: enable default command and help message update #4343

dragon512 merged 1 commit into from
Oct 9, 2018

Conversation

@nqyy
Copy link
Contributor

@nqyy nqyy commented Oct 2, 2018 •
edited
Loading

  1. ArgParser now can take a default command with method void set_default_command(std::string const &cmd).
  2. Update the help message for better output.
  3. The version message is taken out from ArgParser.

@nqyy nqyy force-pushed the argparser_update branch 4 times, most recently from cdcfa89 to 09e2fed Compare October 4, 2018 16:39
@bryancall bryancall added this to the 9.0.0 milestone Oct 5, 2018
@bryancall bryancall added the Core label Oct 5, 2018
@nqyy nqyy mentioned this pull request Oct 8, 2018
Merged
@nqyy nqyy force-pushed the argparser_update branch from 65f2227 to d29ef2b Compare October 8, 2018 19:01
@nqyy nqyy force-pushed the argparser_update branch from 4b5481c to 98380d7 Compare October 9, 2018 19:34
@dragon512 dragon512 merged commit e677ceb into apache:master Oct 9, 2018
@bryancall bryancall modified the milestones: 9.0.0, 8.0.1 Oct 16, 2018
@nqyy nqyy deleted the argparser_update branch November 2, 2018 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dragon512 dragon512 dragon512 approved these changes

Assignees

@nqyy nqyy

Labels

Core

Projects

None yet

Milestone

8.0.1

Development

Successfully merging this pull request may close these issues.

3 participants

@nqyy @dragon512 @bryancall