-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Test mypy integration with pytest #735
Conversation
Codecov Report
@@ Coverage Diff @@
## master #735 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 15 15
Lines 2723 2723
Branches 536 536
=====================================
Hits 2723 2723 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, do you want to remove external-mypy
now?
maybe I'm being dumb, but I thought we could remove |
@samuelcolvin Just pushed with external-mypy removed from makefile and CI |
Change Summary
In preparation for integrating a mypy plugin, I've refactored the mypy tests to be called via
mypy.api.run
through pytest.I see several benefits to this:
Related issue number
@samuelcolvin Given this shouldn't have any effects on external usage of the library, I figured this pull request would be the right place for discussion, but let me know if you'd like me to open a separate issue.
Checklist
changes/<pull request or issue id>-<github username>.rst
file added describing change(see changes/README.md for details)