-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
PR: Remove Python 2 support (part I) #20366
Conversation
Thanks a lot @oscargus for your help with this! It looks pretty good to me, so we'll try to merge it as soon as we release 5.4.2 so that it doesn't get outdated.
Let's leave that for another PR so we can have this one in quickly. |
@oscargus, I'm going to fix the small merge conflict that showed up here and merge your PR so it doesn't get outdated, like last time. |
- Restore missing import and remove unnecessary code. - Forcibly change line endings of a QDarkstyle file after trying to update its subrepo.
d0e3bb7
to
166ab5d
Compare
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.
Thanks a lot @oscargus for your help with this!
Description of Changes
Removing Python 2 support. I may continue with removing
py3compat.py
completely, but will come back to that.Replaces #17173 (will really try to follow this through this time...)
Issue(s) Resolved
Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.
I certify the above statement is true and correct:
oscargus/Oscar Gustafsson