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

markdown/__main__.py: (parse_options): order options alphabetically #1522

Closed
wants to merge 1 commit into from

Conversation

jaalto
Copy link

@jaalto jaalto commented Apr 3, 2025

With -h, --help, order the option listing alphabetically.

@waylan
Copy link
Member

waylan commented Apr 3, 2025

What is the benefit of this change?

Currently, the options are listed logically. For example, --encoding is listed right after --file as it applies to the input file. Similarly, --extension_configs is listed right after --extensions. And then we group together --quiet, --verbose, and --noisy in that logically progressive order. Seems to me that this is much more helpful to users than to list the options alphabetically.

@waylan waylan added the needs-decision A decision needs to be made regarding request. label Apr 3, 2025
@facelessuser
Copy link
Collaborator

I think this is a highly subjective change. I'm personally a -1 on this. I prefer the logical organization.

@waylan
Copy link
Member

waylan commented Apr 9, 2025

I agree that this is subjective. As the maintainers prefer the existing scheme and no persuasive arguments were provided to support the. change, I am closing this as won't fix.

@waylan waylan closed this Apr 9, 2025
@waylan waylan added wontfix The issue will not be fixed for the stated reasons. rejected The pull request is rejected for the stated reasons. and removed needs-decision A decision needs to be made regarding request. labels Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rejected The pull request is rejected for the stated reasons. wontfix The issue will not be fixed for the stated reasons.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants