Skip to content

Conversation

@gus-nx
Copy link
Contributor

@gus-nx gus-nx commented Dec 17, 2025

Allows users to speciify the sorting order via a new option :
order=(asc|desc)
with asc by default (the current behavior)

Copy link
Collaborator

@JeffFaer JeffFaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks for this improvement!

I'm happy to accept this as is, but I believe there might be a little bit more work to do in the future. Regexes support extracting multiple pieces from the current line. Users at some point might want to sort one "column" ascending and another "column" descending, so we might want this option to accept multiple values at some point in the future

Perhaps the behavior would be if there's only one sort order specified, use that for all columns, otherwise require that the number of sort orders matches the number of columns exactly

But we can leave that speculation to the future when someone actually asks for this behavior :)

@JeffFaer JeffFaer merged commit 1605477 into google:main Dec 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants