Skip to content
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

*: Drop support for Python 2 #2443

Merged
merged 28 commits into from
Jul 14, 2018
Merged

*: Drop support for Python 2 #2443

merged 28 commits into from
Jul 14, 2018

Conversation

bdarnell
Copy link
Member

Kick off 6.0 by deleting everything that is now obsolete:

  • Non-asyncio IOLoop and Future implementations
  • Callback arguments
  • stack_context
  • WSGIAdapter
  • Legacy gen interfaces
  • Tests that use exec to work around the absence of async keywords in py2
  • Lots of conditional imports and other mixed 2/3 hackery

Net result: total line count of the repo (including docs, etc. Measured with git ls-files | xargs wc) drops from 57037 to 48829, a reduction of 8208 lines or almost 15%.

Also temporarily remove nightly from the list, because travis's
"nightly" on trusty is pinned to an alpha of 3.7 with some slightly
different behaviors (testing on 3.7 is covered by appveyor)
This was py2-only and is no longer used.
This is the riskiest change in this series since it does some internal
refactoring to get rid of some callback-related machinery.
Just enough to unbreak the build for now.
@bdarnell bdarnell merged commit f6e98e2 into tornadoweb:master Jul 14, 2018
@bdarnell bdarnell deleted the drop-py2 branch July 14, 2018 21:21
@kinow kinow mentioned this pull request Sep 6, 2018
6 tasks
toabctl added a commit to toabctl/opentracing-python that referenced this pull request Mar 11, 2019
tornado >= 6 dropped support for python2 (see [1]) so pin to a lower
version for now.

Fixes opentracing#115

[1] tornadoweb/tornado#2443
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant