Skip to content

Commit

Permalink
Update limits of dependencies after dask test disabling (#22046)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
potiuk committed Mar 7, 2022
1 parent 2ab8f24 commit 082aafd
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
5 changes: 1 addition & 4 deletions setup.cfg
Expand Up @@ -163,10 +163,7 @@ install_requires =
tabulate>=0.7.5
tenacity>=6.2.0
termcolor>=1.1.0
# typing-extensions can be removed under two scenarios: dropping support for python 3.7
# or bumping the minimum version of airflow for providers to 2.2.* which would allow the use of airflow.typing_compat
# Kubernetes Tests also rely on typing-extensions < 4.0.0 - fixing the tests should allow to remove the upperbound
typing-extensions>=3.7.4,<4.0.0
typing-extensions>=3.7.4
unicodecsv>=0.14.1
# Werkzeug is known to cause breaking changes and it is very closely tied with FlaskAppBuilder and other
# Flask dependencies and the limit to 1.* line should be reviewed when we upgrade Flask and remove
Expand Down
3 changes: 3 additions & 0 deletions setup.py
Expand Up @@ -639,6 +639,9 @@ def write_version(filename: str = os.path.join(*[my_dir, "airflow", "git_version
'pytest-xdist',
'python-jose',
'pywinrm',
# The Responses 0.19.0 released on 07.03.2022 break our S3 tests. This limitation should be
# Removed when https://github.com/getsentry/responses/issues/511 is solved.
'responses<0.19.0',
'qds-sdk>=1.9.6',
'pytest-httpx',
'requests_mock',
Expand Down

0 comments on commit 082aafd

Please sign in to comment.