Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC: Put the useful constants first #13696

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

Merged
merged 1 commit into from
Jun 2, 2019

Conversation

eric-wieser
Copy link
Member

Previously the constants at the top of the page said words to the effect of "use the one at the bottom of this page instead".
Let's just show users those ones first to start with.

This patch is a reordering, nothing more.

At some point, I'm tempted to deprecate all of these

Previously the constants at the top of the page said words to the effect of "use the one at the bottom of this page instead".
Let's just show users those ones first to start with.

This commit only re-orders these constants, it does not change any of the descriptions, despite the diff appearing to show that
@charris
Copy link
Member

charris commented Jun 2, 2019

At some point, I'm tempted to deprecate all of these

Why?

@charris charris merged commit 495de50 into numpy:master Jun 2, 2019
@charris
Copy link
Member

charris commented Jun 2, 2019

Thanks Eric.

@eric-wieser
Copy link
Member Author

Why?

Answered in #13705

@eric-wieser eric-wieser deleted the deprecated-constants-last branch June 4, 2019 02:40
@eric-wieser
Copy link
Member Author

Looks like this didn't actually have the intended effect of changing the order in the generated docs. At least the crosslinks are now there.

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

Successfully merging this pull request may close these issues.

2 participants