Use getopt style short/long argument flags (without using getopt package) #34
Conversation
|
@patrislav1 less code but works a little bit differently for short flags |
|
don't have a strong opinion, I mean being able to combine flags is nice but it's not crucial and most of the flags require arguments anyway |
| --help, -h Show help | ||
| --listcolorschemes List color schemes | ||
| --transparent Transparent background | ||
| --width, -w Terminal width (auto if not set) |
There was a problem hiding this comment.
I shorted some texts to stay within 80 characters. Otherwise help text look good? should include something more?
| program | %[1]s > file.svg | ||
|
|
||
| `[1:], fs.Name()) | ||
| fs.VisitAll(func(f *flag.Flag) { |
There was a problem hiding this comment.
Should maybe comment that is a ugly hack to skip the short flags
Mm same :) i usually try to be conservative about not using standard library. But i should probably add some short flag tests at least... and at the same time maybe change the test to use BTW if you want i can add you as a collaborator so you can do more things yourself, i trust your judgement :) |
Thank you 😄 Yes it would make sense then I guess. |
wader
force-pushed
the
flag-short-long
branch
from
062db47 to
fd0e8f5
Compare
Alternative to #30 without using getopt package. One difference is that multiple short flags can't be combined ex:
-abinstead of-a -bdoes not work.