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
Declare raiden-contracts to be typed via PEP561 #813
Conversation
When other packages import raiden-contracts, the type signatures will not be used by mypy unless the package declares itself as properly types. This can be done via PEP561: https://mypy.readthedocs.io/en/latest/installed_packages.html#using-pep-561-compatible-packages-with-mypy Adding this will improve type checking quality in raiden-services (and potential other packages) whenever imports from raiden-contracts are used.
When raiden-network/raiden-contracts#813 gets merged we need this changes to avoid mypy errors.
Codecov Report
@@ Coverage Diff @@
## master #813 +/- ##
=======================================
Coverage 82.82% 82.82%
=======================================
Files 20 20
Lines 1240 1240
Branches 129 129
=======================================
Hits 1027 1027
Misses 182 182
Partials 31 31
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #813 +/- ##
=======================================
Coverage 82.82% 82.82%
=======================================
Files 20 20
Lines 1240 1240
Branches 129 129
=======================================
Hits 1027 1027
Misses 182 182
Partials 31 31
Continue to review full report at Codecov.
|
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.
Looks good to me.
When raiden-network/raiden-contracts#813 gets merged we need this changes to avoid mypy errors.
When other packages import raiden-contracts, the type signatures will
not be used by mypy unless the package declares itself as properly
types. This can be done via PEP561:
https://mypy.readthedocs.io/en/latest/installed_packages.html#using-pep-561-compatible-packages-with-mypy
Adding this will improve type checking quality in raiden-services (and
potential other packages) whenever imports from raiden-contracts are
used.