Skip to content
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.

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

Feature request : Align = and sort options (PR included). #192

Closed
rquadling opened this issue Jan 27, 2023 · 1 comment
Closed

Feature request : Align = and sort options (PR included). #192

rquadling opened this issue Jan 27, 2023 · 1 comment

Comments

@rquadling
Copy link
Contributor

When putting INI files into a repo, having the entries alphabetically sorted vs being unsorted / unordered, helps a little.

The idea is that things exist in some order in many places in codebases. So when you are unfamiliar with a project and see unordered information ... the question that comes to mind is "What on earth is this order???".

The second thing is purely a cosmetic issue where some projects I've worked on like the = to align. Again, common for some languages.

The PR #190 covers both of these features.

I'm not a JS expert, but the tests I've written provide the consistent output.

I also added tests that covered all combinations of the options (simply as permutations matter sometimes).

@wraithgar
Copy link
Member

#205

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

No branches or pull requests

2 participants