search

AcademySoftwareFoundation / rez

An integrated package configuration, build and deployment system for software
https://rez.readthedocs.io
Apache License 2.0
938 stars 332 forks source link

Remove all traces of Python 2 #1635

Open JeanChristopheMorinPerso opened 7 months ago

JeanChristopheMorinPerso commented 7 months ago

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
- [x] Remove `rez.backport`
- [ ] https://github.com/AcademySoftwareFoundation/rez/issues/1646
- [ ] https://github.com/AcademySoftwareFoundation/rez/issues/1648
- [ ] https://github.com/AcademySoftwareFoundation/rez/issues/1637
- [ ] Remove/replace all deprecated (python 2) magic methods (`__<method>__`)
- [ ] https://github.com/AcademySoftwareFoundation/rez/issues/1636
- [ ] Look at PRs/commits that added support for Python 3 and see if we missed anything
- [ ] Run https://github.com/asottile/pyupgrade

If 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. * https://github.com/AcademySoftwareFoundation/rez/pull/736 * https://github.com/AcademySoftwareFoundation/rez/pull/740 * https://github.com/AcademySoftwareFoundation/rez/pull/742 * https://github.com/AcademySoftwareFoundation/rez/pull/745 * https://github.com/AcademySoftwareFoundation/rez/pull/753 * https://github.com/AcademySoftwareFoundation/rez/pull/766 * https://github.com/AcademySoftwareFoundation/rez/pull/733 * https://github.com/AcademySoftwareFoundation/rez/pull/743 * https://github.com/AcademySoftwareFoundation/rez/pull/727 * https://github.com/AcademySoftwareFoundation/rez/pull/755 * https://github.com/AcademySoftwareFoundation/rez/pull/725 * https://github.com/AcademySoftwareFoundation/rez/pull/713 * https://github.com/AcademySoftwareFoundation/rez/pull/712 * https://github.com/AcademySoftwareFoundation/rez/pull/710
hww-nikhilshankarnoble commented 7 months ago

@JeanChristopheMorinPerso has anyone picked up the removal of rez.utils.py23 task yet? I was looking for a straightforward task to kickstart hopefully contributing to Rez more in the future, and would like to pick it up if not :)

JeanChristopheMorinPerso commented 7 months ago

@hww-nikhilshankarnoble I don't think so. I have created https://github.com/AcademySoftwareFoundation/rez/issues/1646 if you want to work on it.