-
-
Notifications
You must be signed in to change notification settings - Fork 6
Add type stubs #56
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
Add type stubs #56
Conversation
32a751d to
24e87cb
Compare
try again clean up 3rd times a charm one more
|
gentle ping for whenever you have a moment to look @MarshalX - I know there's probably more pressing things so I'll check back later 🙂 |
|
That's an amazing job, thank you!! I saw it already but did not comment immediately because this PR also integrates UV. I am still thinking about it but if we want to speed up it let's just include .pyi file :) |
|
i don't think anything (else, after removing EDIT: doh the lockfile! duh, let me remove that |
CodSpeed Performance ReportMerging #56 will degrade performances by 19.95%Comparing Summary
Benchmarks breakdown
|
not sure if related to my changes (i suspect not) or if these just have high variance |
|
@zzstoatzz that's just a noise from small bechmarks. ignore it :) |
|
@zzstoatzz let's give UV a shot! because i do not like this deps list duplication in |
|
thanks a lot!! and huge sorry for the so long delay; tbh it was never what to add except stubs to the new release; but now good bug reports came and has been fixed |
adds a
python/libipldto house the.pyifile and the__all__following the
_{module_name}pattern that I use (and got frompydantic-core), note thatmaturin developknows about themodule-nameset inpyproject.tomllet me know if there are any python type translations I messed up!
before
after