-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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-4505] Correct Tag ALL for PY3 #5275
Conversation
Do I need to base the PR on something else than master ? So the current Pypi on 1.10.3 , work directly when this PR is (I hope) merged |
Codecov Report
@@ Coverage Diff @@
## v1-10-test #5275 +/- ##
==============================================
- Coverage 79.17% 79.16% -0.01%
==============================================
Files 243 243
Lines 17806 17806
==============================================
- Hits 14098 14097 -1
- Misses 3708 3709 +1
Continue to review full report at Codecov.
|
I think we should just remove the PY3 check in master. In master we anyhow only support PY3 now. If we would like to change it in 1.10.x series we should branch it from v1-10-test, but I think there installing devel_ci would do the trick and maybe we should not introduce a new tag just for that in the 1.10.x series. |
@potiuk Thank you , I just changed the base of the PR to v1-10-test. Yes devel_ci do the trick , but that's what I was arguing against in the issue of the PR : It's not very user friendly. |
OK. I looked back at it, and I have second thought. Would not it be better to simply make devel_all conditional based on PY3 same way devel_ci is done? that would not introduce new tag, and it would magically work if you run it in py3 virtualenv :) @raphaelauv . |
4df2044
to
c6f5bd9
Compare
@potiuk It could, but for people currently using devel_all with PY2 it would be a regression , is this a risk you want a take ? |
@raphaelauv What regression ? I cannot see any scenario where it could break things ? If you are using py2 it will continue working as it did before. It's just py3 will not install snakebite . |
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.
That's much nicer :)
Should this be merged now that it's approved? |
Make sure you have checked all steps below.
Jira
Description
A new tag , so we could get ALL PY3 packages ( without PY2 )
Tests
Commits
Documentation
Code Quality
flake8