rad issue list –sort=
and
rad issue list --rsort=<mode>
to sort the issue listing.
rad issue list –sort=
and
rad issue list --rsort=<mode>
to sort the issue listing.
I like the addition of sorting! However, sorting by comment count seems dubious to me. If I was to think sort by comments, my first instinct would be to sort by latest comments, i.e. which issues were commented on most recently.
Rebased.
Nice, thanks for the patch! I have to major points (and left a few inline comments):
- I’d prefer to rename the flag to
sort-by(and in the help, call the valuefield). I think this stays more with the concepts and wording around databases - I’d also make it possible to sort by timestamp (
--sort-by date), such that I can have the oldest ones on top with--rsort-by date
Also, could you please reword the commit message such that it uses a component prefix, here cli: ...?
NOTE: Please also check the comments on the initial revision :)
I think we could keep the comment count, if we would allow “piping” args, e.g. to find old issues with little attention you could do --rsort-by date --sort-by comments? Please note, that every issue at the least 1 top-level root comment.
Otherwise, I’m also fine with removing the comment count.
I like --sort-by, but at the same time I would like to suggest to reconsider --rsort and think about alternatives like --reverse, as we are used from git log, or maybe a flag --sort-descending.
Rebased.