Eric Greer

Results 114 comments of Eric Greer

The `StringSlice` flag was already modified to split on commas, but we may want to revert that if we want commas to remain literals as originally designed: https://github.com/integrii/flaggy/commit/b5bd28929feea25250a7b6c411844d54eda34b99

I think we could keep the default comma parsing behavior here while adding a new string slice flag type that takes commas directly without considering them a character to split...

Oof. I agree that does not look great. We could put some logic in to determine how many tabs to use when displaying help output like that to fix things...

Currently you can show help on a `flaggy.Parser`, but not a `flaggy.Subcommand`. I agree that it would be nice to call help on a `flaggy.Subcommand` directly, but it would take...

This looks really useful and should be do-able by simply adding some more metadata to our flags and editing the default help template.

I am not sure if how flags are listed is **always** the same (or always based on the code), actually, but we could add sorting to them - it's probably...

We could add a couple calls to flaggy that enable flag output sorting. Then, right before output, we sort depending on which sorting method was chosen. ``` flaggy.SortFlagsByLongName() flaggy.SortFlagsByLongNameReversed() ......

I am leaning toward calling this "out of scope" for flaggy because its so radically different than the currently patterns. I think [go-flags](https://github.com/jessevdk/go-flags) already does a pretty good job of...

Thanks Kost. I am going to review this tonight - sorry for the delay! I really appreciate the contribution.

We definitely want to get a test or two in here to validate the functionality. I would also like to see more functions and flatter code in this - that's...