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

Set a floor for versions of typing_extensions #518

Merged
merged 1 commit into from
Feb 11, 2022
Merged

Conversation

sirosen
Copy link
Member

@sirosen sirosen commented Feb 11, 2022

typing_extensions>=4 is required (where typing_extensions is required).

This is tested in tox both with mypy-mindeps and with py36-mindeps. The mypy run is only done in GH Actions to reduce the time spent on this when running make lint, as it is unlikely to fail.

The typing repo has a discussion about capping or not capping the version of typing_extensions, but it is not well-resolved. For now, do not apply a cap to the version of typing_extensions. We can revisit this if the lack of a cap ever becomes an issue.

`typing_extensions>=4` is required (where typing_extensions is
required).

This is tested in tox both with `mypy-mindeps` and with
`py36-mindeps`. The mypy run is only done in GH Actions to reduce the
time spent on this when running `make lint`, as it is unlikely to
fail.

The `typing` repo has a discussion about capping or not capping the
version of `typing_extensions`, but it is not well-resolved. For now,
do not apply a cap to the version of `typing_extensions`. We can
revisit this if the lack of a cap ever becomes an issue.
@sirosen sirosen merged commit cd0908d into globus:main Feb 11, 2022
@sirosen sirosen deleted the bound-te branch February 11, 2022 20:53
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.

2 participants