search

psyinfra / onyo

text-based inventory system on top of git
ISC License
3 stars 5 forks source link

BF: make sorting case sensitive #652

Closed aqw closed 2 months ago

aqw commented 2 months ago

Previously, sorting was case insensitive, so that the words would actually be grouped together. However, within a word, it was not sorted.

GROUPLETTERS does not do what we want, but using LOCALE does (at least with 'en_US.UTF-8').

For now, I am choosing to use whatever the user has configured on their OS. That makes sense to me, respecting their choice. If this ends up biting us in practice, then we'll simply hardcode it or make it a configurable option in .onyo/config.

fix #651

aqw commented 2 months ago

RTD failure is unrelated.

aqw commented 2 months ago

I filed #653 to unblock that task from this PR.

aqw commented 2 months ago

+1 from Ben on Matrix. Merging!