KEMBAR78
gh-58032: Docs: Sort argparse alphabetically by hugovk · Pull Request #125871 · python/cpython · GitHub
Skip to content

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented Oct 23, 2024

Follow on from #124664, let's keep the modules sorted alphabetically.


📚 Documentation preview 📚: https://cpython-previews--125871.org.readthedocs.build/

Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

I intentionally moved builtins at the beginning of the list.

Copy link
Member

@savannahostrowski savannahostrowski left a comment

Choose a reason for hiding this comment

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

Thanks Hugo!

Copy link
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

🚢

@willingc
Copy link
Contributor

I intentionally moved builtins at the beginning of the list.

@serhiy-storchaka What prompted you to move to the beginning? Perhaps we need a sentence or section break between them @hugovk

@serhiy-storchaka
Copy link
Member

Builtins look special to me, and in some other places we list them separately from other modules. I thought that even if we list them in the common list with other modules, it is worth to move the builtins module to the beginning. But I may be wrong here -- in few last What's New files (3.13, 3.14) they are listed alphabetically in the list of modules.

@willingc
Copy link
Contributor

Thanks for clarifying @serhiy-storchaka. @hugovk, do you want to offset built-ins from the rest of standard library modules?

@hugovk
Copy link
Member Author

hugovk commented Oct 25, 2024

In a way it's just another module in the docs, and I'd prefer alphabetical, but I don't mind too much.

If we're browsing a long list, it's easier to find alphabetically, and it'll be less likely for new edits to be added in arbitrary order if it begins with A instead of B.

cc @AA-Turner who edited the last two versions, any opinion?

@willingc
Copy link
Contributor

I'm going to go ahead and merge this to unblock the PR. We can always create future iterations to improve the difference between builtins and the other libraries.

@willingc willingc merged commit d4b6d84 into python:main Oct 30, 2024
25 checks passed
@hugovk hugovk deleted the docs-argparse-sort branch October 30, 2024 06:40
picnixz pushed a commit to picnixz/cpython that referenced this pull request Dec 8, 2024
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants