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

Jump to bottom

gh-91340: Document multiprocessing.set_start_method force parameter #32339

Merged
merged 6 commits into from Nov 27, 2022
Merged

gh-91340: Document multiprocessing.set_start_method force parameter #32339

merged 6 commits into from Nov 27, 2022

Conversation

dignissimus
Copy link
Contributor

@dignissimus dignissimus commented Apr 5, 2022 •
edited by miss-islington

#91340

https://bugs.python.org/issue47184

Automerge-Triggered-By: GH:kumaraditya303

@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Apr 5, 2022
@dignissimus dignissimus marked this pull request as ready for review Apr 11, 2022
@dignissimus dignissimus changed the title bpo-47184: Document multiprocessing.set_start_method force parameter gh-91340: Document multiprocessing.set_start_method force parameter Apr 14, 2022
@MaxwellDupre
Copy link
Contributor

MaxwellDupre commented Apr 23, 2022

Where you have
method can be 'fork', 'spawn' or 'forkserver'.
The start of the sentence is a lower case italic (in the middle of a line of text), but it looks strange: optionally start new line or make more distinctive, please.

@johnthagen johnthagen mannequin mentioned this pull request Apr 10, 2022
Closed
@kumaraditya303 kumaraditya303 added needs backport to 3.10 needs backport to 3.11 🤖 automerge PR will be merged once it's been approved and all CI passed labels Nov 27, 2022
@miss-islington
Copy link
Contributor

miss-islington commented Nov 27, 2022

Status check is done, and it's a success ✅.

@miss-islington miss-islington merged commit dfc2732 into python:main Nov 27, 2022
14 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Nov 27, 2022

Thanks @dignissimus for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

miss-islington commented Nov 27, 2022

Sorry, @dignissimus, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker dfc2732a57e3ea6603d62f769d4f9c80be726fa4 3.10

@bedevere-bot
Copy link

bedevere-bot commented Nov 27, 2022

GH-99820 is a backport of this pull request to the 3.11 branch.

@miss-islington miss-islington self-assigned this Nov 27, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 27, 2022
@dignissimus @miss-islington
pythongh-91340: Document multiprocessing.set_start_method force param…
ba15346 
…eter (pythonGH-32339)

pythonGH-91340

https://bugs.python.org/issue47184
(cherry picked from commit dfc2732)

Co-authored-by: Sam Ezeh <sam.z.ezeh@gmail.com>
Automerge-Triggered-By: GH:kumaraditya303
@kumaraditya303
Copy link
Contributor

kumaraditya303 commented Nov 27, 2022

Thanks.

miss-islington added a commit that referenced this pull request Nov 27, 2022
@miss-islington @dignissimus
gh-91340: Document multiprocessing.set_start_method force parameter (G…
366adc7 
…H-32339)

GH-91340

https://bugs.python.org/issue47184
(cherry picked from commit dfc2732)

Co-authored-by: Sam Ezeh <sam.z.ezeh@gmail.com>
Automerge-Triggered-By: GH:kumaraditya303
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kumaraditya303 kumaraditya303

Assignees

@miss-islington miss-islington

Labels
docs Documentation in the Doc dir expert-multiprocessing needs backport to 3.10 🤖 automerge PR will be merged once it's been approved and all CI passed skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@dignissimus @MaxwellDupre @miss-islington @bedevere-bot @kumaraditya303 @vstinner @the-knights-who-say-ni @ezio-melotti @JelleZijlstra