Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Require explicit Optional in type annotations #79

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

hukkin
Copy link
Contributor

@hukkin hukkin commented Nov 5, 2020

This PR disables the "implicit optional" mypy feature. Type annotations relying on it are no longer PEP 484 compliant (see python/peps#689). Mypy will also turn the feature off in a future release (see python/mypy#9091).

@prusnak prusnak requested a review from matejcik November 5, 2020 08:18
@matejcik
Copy link
Contributor

matejcik commented Nov 5, 2020

😿

@hukkin
Copy link
Contributor Author

hukkin commented Nov 5, 2020

@matejcik Would it be possible to make a PyPI release? Would be cool to have all the annotations brought out 😸

@matejcik matejcik merged commit e63ced2 into trezor:master Jul 27, 2021
@matejcik
Copy link
Contributor

this is now out on pypi

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants