New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add option to make alphabetical sort decorator case-sensitive #6

Open

TaylorKrusen opened this issue Oct 31, 2023 · 0 comments

Labels

decorator enhancement good first issue

Contributor

TaylorKrusen commented Oct 31, 2023 •

edited

Loading

PR #5 added an example of a decorator for alphabetical sorting. Someone highlighted the importance of case-sensitivity.

Here's a paraphrased example of sorting challenges caused by casing:

Pretend you're looking for the term "API Keys" on an alphabetically sorted list. Which order would you expect?

Case sensitive Not case-sensitive

AML
API keys
Accounts
Allowlists
Application owners
Application users
Coupons Accounts
Allowlists
AML
API keys
Application owners
Application users
Coupons

The redocly-cli solved for this using an ignoreCase flag. A similar solution might work well here.

TaylorKrusen added enhancement good first issue decorator labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment