Skip to content

[3.6] bpo-32208: update threading.Semaphore docs and add unit test (GH-4709) #4750

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.

Sign up for GitHub

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
Merged
merged 1 commit into from
Dec 7, 2017
Merged

[3.6] bpo-32208: update threading.Semaphore docs and add unit test (GH-4709) #4750

merged 1 commit into from
Dec 7, 2017

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Dec 7, 2017 •
edited by bedevere-bot
Loading

  • fix issue32208: update threading.Semaphore docs and add unit test to validate correct behavior

  • add test for blocking

  • Update threading.rst

  • semaphore: remove documentation validation tests and move 'return value' test to BaseSemaphore
    (cherry picked from commit a0374dd)

https://bugs.python.org/issue32208

@vitiral @miss-islington
bpo-32208: update threading.Semaphore docs and add unit test (pythonG…
70b5d1c 
…H-4709)

* fix issue32208: update threading.Semaphore docs and add unit test to validate correct behavior

* add test for blocking

* Update threading.rst

* semaphore: remove documentation validation tests and move 'return value' test to BaseSemaphore
(cherry picked from commit a0374dd)
@bedevere-bot bedevere-bot mentioned this pull request Dec 7, 2017
Merged
@miss-islington
Copy link
Contributor Author

@vitiral and @asvetlov: Backport status check is done, and it's a success ✅ .

@asvetlov asvetlov merged commit a04ca12 into python:3.6 Dec 7, 2017
@miss-islington
Copy link
Contributor Author

Thanks, @asvetlov!

@miss-islington miss-islington deleted the backport-a0374dd-3.6 branch December 7, 2017 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@miss-islington @asvetlov @the-knights-who-say-ni @bedevere-bot @vitiral