-
Notifications
You must be signed in to change notification settings - Fork 242
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
Deprecate methods from threading module #3839
Conversation
@YongxueHong @lkotek @yanan-fu would you mind to help review this one? thanks in advance! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm with my eyes that the effected code about the deprecated threading method are all updated by this patch.
Ack
For the python 3.7 checking failure, as avocado have drop the python 3.7 with :avocado-framework/avocado@a17bfff Do we need to update the ci accordingly in avocado-vt with a separate commit or patch now ? avocado-vt/.github/workflows/ci.yml Line 37 in 9485fc6
|
Eh~ I missed #3832 and it just be merged. So python 3.7 CI checking failure issue has gone . |
There are a few threading methods been deprecated since Python3.10, let's replace them according to the guide. Reference: https://docs.python.org/3/whatsnew/3.10.html#deprecated Issue: avocado-framework#3837 Signed-off-by: Xu Han <[email protected]>
1a56744
to
e82b82c
Compare
I checked with the python docs, I don't see any issue here. Thanks for the patch! Ack |
Thanks to all! |
There are a few threading methods been deprecated since Python3.10, let's replace them according to the guide.
Reference: https://docs.python.org/3/whatsnew/3.10.html#deprecated
Issue: #3837
ID: 1791