Skip to content

Conversation

@SebastiaanZ
Copy link
Contributor

@SebastiaanZ SebastiaanZ commented Jul 22, 2023

I followed the backport PR instructions, as this PR is made against a different branch than main, but it's not a backport in the sense that the changes are not relevant for main (the text in the docs has been removed in 3.12). The original issue, #106975, mentions that 3.11 should be the target branch.


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

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Thanks! We'll also backport this to 3.10 to fix https://docs.python.org/3.10/library/random.html#random.randrange

@hugovk hugovk added needs backport to 3.10 only security fixes and removed awaiting merge labels Jul 22, 2023
@hugovk hugovk merged commit 4f7c23e into python:3.11 Jul 22, 2023
@miss-islington
Copy link
Contributor

Thanks @SebastiaanZ for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-106987 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jul 22, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 22, 2023
…6975) (pythonGH-106985)

(cherry picked from commit 4f7c23e)

Co-authored-by: Sebastiaan Zeeff <33516116+SebastiaanZ@users.noreply.github.com>
@hugovk
Copy link
Member

hugovk commented Jul 22, 2023

Thanks! We'll also backport this to 3.10 to fix docs.python.org/3.10/library/random.html#random.randrange

Correcting myself: 3.10 is security-only so we don't backport this (https://devguide.python.org/versions/). And https://docs.python.org/ points to 3.11 so that's the main thing. Thanks!

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