-
Notifications
You must be signed in to change notification settings - Fork 100
Add type hinting #148
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
Comments
Love using the library, works awesome. Question though. Would it be possible to generate an array of questions given a dataclass as input? Or is that way out of scope for this library? |
As mypy states: Skipping analyzing "inquirer": found module but no type hints or library stubs. |
@Lonerider2010 0% done unfortunatelly. You have options:
|
Sorry, but I think it is not going to be so easy, because there are mutable
types: if you pass a string as default, it will be used, but if you pass a
callable it will be called.
As far as I remember that happens with ar least 2 or 3 different fields.
…--
Sorry for being so brief, but I'm writing you from my mobile.
El jue., 23 jun. 2022 21:32, staticdev ***@***.***> escribió:
@Lonerider2010 <https://github.com/Lonerider2010> 0% done unfortunatelly.
You have options:
1. Add to mypy ignore (as I used to do)
2. Use a similar typed library (such as prompt-toolkit)
3. Contribute with a PR and I will be glad to review it
—
Reply to this email directly, view it on GitHub
<#148 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEOSWWXVWK5YGQIDQDLVN3VQS3WVANCNFSM5K4VRYUQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
@magmax Thanks for your reply. According to mutable types: What about Union? Or if nothing else helps, there is still the type hint Any... |
BTW. the answer to @jayrod's question is of great interest for me, too. |
im working on it |
almost done with a lot of the types. some parts are erroring in the strictest setting but thats because i dont know how to do generics and type inference in python that well. (im too TS pilled) the normal setting has very few problems |
The text was updated successfully, but these errors were encountered: