Skip to content

Fix Documentation for PyType_GetFlags #10758

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 2 commits into from
Nov 29, 2018
Merged

Fix Documentation for PyType_GetFlags #10758

merged 2 commits into from
Nov 29, 2018

Conversation

eduardo-elizondo
Copy link
Contributor

Trivial change, no need for bug report or NEWS

@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Nov 28, 2018
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.

Please document that it was changed in 3.4 by adding the corresponding versionchanged directive.

@serhiy-storchaka
Copy link
Member

serhiy-storchaka commented Nov 29, 2018

For reference: bpo-16086.

@eduardo-elizondo
Copy link
Contributor Author

eduardo-elizondo commented Nov 29, 2018

@serhiy-storchaka I didn't know how much explanation you wanted in the versionchanged directive so I went with another example that I've seen in the docs. Just let me know if this is ok or if I should add more context such as:

"The return type is now unsigned long rather than long to allow setting the most significant bit."

@vstinner vstinner merged commit 9fbcfc0 into python:master Nov 29, 2018
@miss-islington
Copy link
Contributor

Thanks @eduardo-elizondo for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-10789 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 29, 2018
PyType_GetFlags() return type is unsigned long, not long.
(cherry picked from commit 9fbcfc0)

Co-authored-by: Eddie Elizondo <eduardo.elizondorueda@gmail.com>
@bedevere-bot
Copy link

GH-10790 is a backport of this pull request to the 3.6 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 29, 2018
PyType_GetFlags() return type is unsigned long, not long.
(cherry picked from commit 9fbcfc0)

Co-authored-by: Eddie Elizondo <eduardo.elizondorueda@gmail.com>
vstinner pushed a commit that referenced this pull request Nov 29, 2018
PyType_GetFlags() return type is unsigned long, not long.
(cherry picked from commit 9fbcfc0)

Co-authored-by: Eddie Elizondo <eduardo.elizondorueda@gmail.com>
vstinner pushed a commit that referenced this pull request Nov 29, 2018
PyType_GetFlags() return type is unsigned long, not long.
(cherry picked from commit 9fbcfc0)

Co-authored-by: Eddie Elizondo <eduardo.elizondorueda@gmail.com>
@vstinner
Copy link
Member

Thanks @eduardo-elizondo, I merged your change in master, and backported it to 3.6 and 3.7.

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 issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants