Skip to content

Fix Documentation for PyType_GetFlags#10758

Merged
vstinner merged 2 commits intopython:masterfrom
eduardo-elizondo:getflags-docs
Nov 29, 2018
Merged

Fix Documentation for PyType_GetFlags#10758
vstinner merged 2 commits intopython:masterfrom
eduardo-elizondo:getflags-docs

Conversation

@eduardo-elizondo
Copy link
Contributor

@eduardo-elizondo eduardo-elizondo commented Nov 28, 2018

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
serhiy-storchaka
serhiy-storchaka reviewed Nov 29, 2018
View reviewed changes
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 •
edited by bedevere-bot
Loading

For reference: bpo-16086.

@eduardo-elizondo
Copy link
Contributor Author

eduardo-elizondo commented Nov 29, 2018 •
edited
Loading

@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

miss-islington commented Nov 29, 2018

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

bedevere-bot commented Nov 29, 2018

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
@eduardo-elizondo @miss-islington
bpo-16086: Fix PyType_GetFlags() documentation (pythonGH-10758)
81a0e28 
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

bedevere-bot commented Nov 29, 2018

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
@eduardo-elizondo @miss-islington
bpo-16086: Fix PyType_GetFlags() documentation (pythonGH-10758)
53b9727 
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
@miss-islington @eduardo-elizondo @vstinner
bpo-16086: Fix PyType_GetFlags() documentation (GH-10758) (GH-10789)
e754159 
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
@miss-islington @eduardo-elizondo @vstinner
bpo-16086: Fix PyType_GetFlags() documentation (GH-10758) (GH-10790)
2a852a2 
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

vstinner commented Nov 29, 2018

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

Reviewers

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes

@vstinner vstinner Awaiting requested review from vstinner

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@eduardo-elizondo @serhiy-storchaka @miss-islington @bedevere-bot @vstinner @the-knights-who-say-ni