Closed orestisfl closed 6 years ago
Looks good! If it was just for me, I would be fine with just location-based sorting, but if there is a use-case for a number-based sorting we can add it.
With the boolean options, we do not have much choice here. I quite like the sort-by
option. An alternative would be to have: all
, all-by-num
, all-by-location
. But this might be confusing.
Thanks for the corrections, up
Then this is ready for merging! Thanks for all the contributions :)
Using
--sort-by <method>
here could be a bit weird since the other boolean arguments (-a, -c, -i, -w) are not parsed with strings. What do you think?We could also add a flag to prioritize the 'primary' output.