-
-
Notifications
You must be signed in to change notification settings - Fork 935
feat(backend): add typechecking with ty #9664
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
base: master
Are you sure you want to change the base?
feat(backend): add typechecking with ty #9664
Conversation
✅ Deploy Preview for inventree-web-pui-preview canceled.
|
…feat(backend)--Add-typechecking-with-ty
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9664 +/- ##
==========================================
- Coverage 86.35% 86.29% -0.06%
==========================================
Files 1229 1229
Lines 53966 54070 +104
Branches 2259 2259
==========================================
+ Hits 46601 46660 +59
- Misses 6793 6838 +45
Partials 572 572
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@wolflu05 would appreciate a look and feedback on the approach |
Seems like only a small tool change. Looks good in general. The only I idea I have regarding mixins is to use the cast function from typing, but I am not sure if thats a good solution long term. |
Adds type checking via astral's (experimental) ty which is pretty fast.
This is technically done but I will keep it as draft till we have 1.0 out to stop an accidential merge