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.

Sign up for GitHub

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

Jump to bottom

Introduce mypy in the tests #82

Closed

lmazuel opened this issue Feb 15, 2019 · 1 comment

Member

lmazuel commented Feb 15, 2019 •

edited

Loading

This issue is about introducing mypy as part of the tox run, for both vanilla and azure. This means:

tox should install mypy as a dev dependency
tox should run mypy before doing the unittests

mypy should be run on both the generated code and the unittests (both Expected and AcceptanceTests folder)

Build should be clean with no errors. This might require update of annotations in msrest and msrestazure to solve some of them.
https://github.com/Azure/msrest-for-python/
https://github.com/Azure/msrestazure-for-python/

This was referenced Feb 15, 2019

Full type hints in models_py3 and aio folder #83

Closed

Generate type hints for Python code #79

Closed

lmazuel added the Autorest v3 refactoring label

Member Author

lmazuel commented Jan 6, 2020

Fixed in #288

lmazuel closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment