-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
PyCharm IDE warnings #2932
Comments
The code is fine, the IDE is not. Python is a very dynamic language and we use that to our purpose in peewee. |
I have previously seen your posts saying that you are against type hinting in python in recent versions, there are big improvements in terms of type hinting (for example, at the moment, I have found a workaround in the form of overriding the get* methods in my BaseModel, something like here:
the methods what is your current opinion on typing? perhaps this is planned in peewee 4.x? |
The problem of |
When I want types I reach for a statically-typed language. |
Hmm I just checked one of the previous ones, and everything seems to be working fine there. Apparently it was broken in the last one
Have you ever wanted to take advantage of both types of languages? I'm interested in your opinion as a professional, I understand that adding type hinting to peewee is a huge job, and maybe you're saying that because you don't want to or you don't have time do it. Suppose you had recently started doing a similar project in Python, would you use type hints? |
Popularity, however you quantify it, isn't a meaningful metric for me. Programming goes through fads. Type hinting, for me, is bad for a couple reasons -- 1) it's not enforced, 2) I find it noisy and distracting, 3) well-formatted docstrings can accomplish the same thing and were the de-facto way to indicate types along w/other useful information and are compatibe with all versions of Python. I have a strong distaste for this kind of code. If I want typing I use go or c.
Never. |
Hello, im using Pycharm with latest peewee 3.17.6
I trying using playhouse and IDE throws error
Also I trying using this code IDE throws warning
Since this code is quite working (
playhouse
too), I think these are errors on the part of the IDE, but is it possible to fix them on your part?The text was updated successfully, but these errors were encountered: