-
Notifications
You must be signed in to change notification settings - Fork 337
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
Remove all traces of Python 2 #1635
Comments
@JeanChristopheMorinPerso has anyone picked up the removal of |
@hww-nikhilshankarnoble I don't think so. I have created #1646 if you want to work on it. |
@JeanChristopheMorinPerso has anyone tackled these magic methods yet?
I may look into that as part of the Dev Days 2024. |
Hi @lbreede . I don't think anyone did that. Feel free to give it a try. But note that rez is not participating to the dez days and as such we might not be able to review your contributions in time. We still appreciate all contributions, so please do contribute if you can. We might be slow to review though. |
@JeanChristopheMorinPerso, I may just tackle it regardless. Is there an issue for this specific task yet? |
Support for Python 2 was removed in 3.0.0, but we left some leftovers. This issue lists what's left to be done to really remove all traces of Python 2 from the codebase.
Tasks
rez.utils.py23
#1646rez.vendor.enum
with the built-in enum module #1648rez.utils.data_utils.cached_property
withfunctools.cached_property
#1637__<method>__
) #1833from __future__ import print_function
from the code base #1636If you want to start working on one of these task and it's not yet converted to an issue, please comment in the issue and we will happily create an issue.
List of Python 2 to 3 PRs
Note that there is a lot of noise in these PRs. But they can be used to see what kind of compatibility layers were added to support both python 2 and 3. This will be handy to determine if something should be removed.
The text was updated successfully, but these errors were encountered: