-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
[AIRFLOW-5251] add missing typing-extensions dep for py37 #5857
Conversation
Oh yes, Protocol isn't in core until 3.8, and even then it's under Could you update the build matrix so that one of the jobs runs on Python 3.7 please? And do you think it is worth keeping this to
possibly bundled up into a module somewhere in the airflow tree so that we can, for example, just do |
Sure, I will make that change today :) |
55a7c78
to
f9f0302
Compare
I have changed the mysql build to use python 3.7. @ashb @potiuk let me know if you prefer to create a new build matrix entry for 3.7 instead. one surprise is some of the identity comparison behaviors have been changed in 3.7, so I had to put in python version check in unit test to change the assertion statement conditionally. |
8f0fd1b
to
493b4ea
Compare
Thanks @houqp (for this and the related PR). Once this is green I'll merge. |
It seems this may be causing a problem with alembic. Since you run alembic from That's probably confusing, but basically if I run If I run from repo root (specifying config file and changing |
Make sure you have checked all steps below.
Jira
Description
Turns out Protocol is still experimental and not available in python 3.7's typing module. So typing-extensions is still required for py3.7.
Tests
Commits
Documentation
Code Quality
flake8