Henry Schreiner
Henry Schreiner
Quick comment; that's the wrong way to run the parse. It should be: ```cpp try { app.parse(argc, argv); } catch (const CLI::ParseError &e) { return app.exit(e); } ``` (or use...
Ahh, yes, you call `get_options` twice, so the begin and end are incorrect. I would recommend using C++11 style iteration if you are in C++17, not C++98. The copy is...
I wouldn't be against adding it if it can be added safely, and if there is a little warning about using it in the docs, saying something like this should...
This is actually rather nicely symmetric with how we currently handle multiple option names. I think I'd be fine with it if it didn't add much complexity.
Does anything need to be checked here (such as for summing) or can it be closed?
A few thoughts: * Following Google's style guide entirely is not a requirement, a subset or collection of deviations would be fine * As much as possible should be checked...
I like option 1 with spdx. And the license file does get included in the single header version.
That was mostly there since older versions were hosted separately, and as such were not linked to the current version like this one is. That was actually the version of...
Wow, haven't seen fixed width full alignment before. I guess one of the next things to work on is proper docs for the formatter. You can replace parts or the...
Does `--option1` *require* an argument? If so, then `--help` is a valid argument. How else would you give an option with two dashes (or, more common, one dash, like, `-1`)?...