X Tutup
The Wayback Machine - https://web.archive.org/web/20221223111451/https://github.com/python/cpython/pull/24642
Skip to content
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

[3.7] bpo-43293: Doc: move note about GIL to top of threading module (GH-24622) #24642

Merged
merged 1 commit into from Feb 26, 2021

Conversation

quantum5
Copy link
Contributor

@quantum5 quantum5 commented Feb 24, 2021

The note about the GIL was buried pretty deep in the threading documentation,
and this made it hard for first time users to discover why their attempts
at using threading to parallelize their application did not work.

In this commit, the note is moved to the top of the module documentation for
visibility.
(cherry picked from commit 32181be)

Co-authored-by: Guanzhong Chen quantum2048@gmail.com

https://bugs.python.org/issue43293

…ythonGH-24622)

The note about the GIL was buried pretty deep in the threading documentation,
and this made it hard for first time users to discover why their attempts
at using threading to parallelizing their application did not work.

In this commit, the note is moved to the top of the module documention for
visibility..
(cherry picked from commit 32181be)

Co-authored-by: Guanzhong Chen <quantum2048@gmail.com>
@bedevere-bot bedevere-bot added skip news docs Documentation in the Doc dir labels Feb 24, 2021
@pablogsal pablogsal merged commit 30927fa into python:3.7 Feb 26, 2021
4 checks passed
@bedevere-bot
Copy link

bedevere-bot commented Feb 26, 2021

@pablogsal: Please replace # with GH- in the commit message next time. 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.

None yet

4 participants
X Tutup