-
Notifications
You must be signed in to change notification settings - Fork 14.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
Update limits of dependencies after dask test disabling #22046
Conversation
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
I think some tests will fail - let's see 🤞 |
Not exactly those that I expected :) but errors they are :) |
Fixing :) |
Some of the tests failed previously with typing extensions above 4. This PR attempts to relax the limit and check if the problems still appear. Also new tests (S3) started to fail when a new `responses` library version has been released today. So this change also add limits to the responses library in order to make sure the tests pass. Issue getsentry/responses#511 has been opened to raise it to `responses` library maintainers.
8bb092f
to
c89909f
Compare
Hey @uranusjr - you might want to re-review this one. Turned out that the errors are coming from (released 5 hrs ago) 0.19.0 version of |
I just pushed the constraint fix (did not push it previously) so it should be good to merge. |
Some of the tests failed previously with typing extensions above 4.
This PR attempts to relax the limit and check if the problems
still appear.
Also new tests (S3) started to fail when a new
responses
libraryversion has been released today.
So this change also add limits to the responses library in order
to make sure the tests pass.
Issue getsentry/responses#511 has been
opened to raise it to
responses
library maintainers.^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.