search

terrastruct / ci

Terrastruct's shared CI
ISC License
6 stars 1 forks source link

flag.sh: Add ability parse combined short flags #6

Open nhooyr opened 1 year ago

nhooyr commented 1 year ago

like -bfzs

Should be parsed as 4 separate flags, not one whole bfzs.

I'm thinking I add a function to iterate through each character somehow and if they don't recognize one of the characters as a flag or it wasn't a single hyphen flag, then we error out.

nhooyr commented 1 year ago

Maybe flag.sh should be changed to use getopts in combination. Perhaps long flags could be supported by a separate optstring.

This would also solve parsing of flags like -oarg where there is no separator between the flag and the argument.