Skip to content

Conversation

@cdce8p
Copy link
Collaborator

@cdce8p cdce8p commented Jan 31, 2024

Implement type application for callables with TypeVar defaults.
Similar to previous PRs, support for TypeVarTuples is still TODO.

Ref: #14851

@github-actions
Copy link
Contributor

Diff from mypy_primer, showing the effect of this PR on open source code:

ibis (https://github.com/ibis-project/ibis)
- ibis/expr/operations/tests/test_generic.py:56: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:62: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:65: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:66: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:67: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:69: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:73: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:75: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:79: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:85: error: Type application has too few types (2 expected)  [misc]
- ibis/expr/operations/tests/test_generic.py:88: error: Type application has too few types (2 expected)  [misc]

@JelleZijlstra JelleZijlstra merged commit 55247c4 into python:master Jan 31, 2024
@cdce8p cdce8p deleted the TypeVar-08-apply-callable branch January 31, 2024 16:10
@cdce8p cdce8p added the topic-pep-696 TypeVar defaults label Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic-pep-696 TypeVar defaults

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants