X Tutup
The Wayback Machine - https://web.archive.org/web/20221223191415/https://github.com/python/cpython/pull/99963
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-66285: remove redundant time.sleep from test_fork_signal_handling #99963

Merged
merged 1 commit into from Dec 3, 2022
Merged

GH-66285: remove redundant time.sleep from test_fork_signal_handling #99963

merged 1 commit into from Dec 3, 2022

Conversation

kumaraditya303
Copy link
Contributor

@kumaraditya303 kumaraditya303 commented Dec 3, 2022 •
edited by miss-islington

Automerge-Triggered-By: GH:kumaraditya303

@bedevere-bot bedevere-bot added awaiting core review tests Tests in the Lib/test dir labels Dec 3, 2022
@kumaraditya303 kumaraditya303 marked this pull request as ready for review Dec 3, 2022
@gvanrossum
Copy link
Member

gvanrossum commented Dec 3, 2022

Could you link this to the original issue (GH-66285)?

@kumaraditya303 kumaraditya303 changed the title remove redundant time.sleep from test_fork_signal_handling GH-66285: remove redundant time.sleep from test_fork_signal_handling Dec 3, 2022
@bedevere-bot bedevere-bot mentioned this pull request Dec 3, 2022
Closed
@kumaraditya303
Copy link
Contributor Author

kumaraditya303 commented Dec 3, 2022

Could you link this to the original issue (GH-66285)?

done

@kumaraditya303 kumaraditya303 added the 🤖 automerge PR will be merged once it's been approved and all CI passed label Dec 3, 2022
@miss-islington miss-islington merged commit ee60156 into python:main Dec 3, 2022
13 checks passed
@kumaraditya303 kumaraditya303 deleted the sleep branch Dec 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gvanrossum gvanrossum

@1st1 1st1

@asvetlov asvetlov

Assignees
No one assigned
Labels
expert-asyncio 🤖 automerge PR will be merged once it's been approved and all CI passed skip news tests Tests in the Lib/test dir
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@kumaraditya303 @gvanrossum @bedevere-bot @miss-islington
X Tutup