-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Clarify documentation for typing
extension
#4549
Conversation
pylint/extensions/typing.py
Outdated
"3.7 - 3.9" | ||
"Set to ``no`` if the app / library does NOT need to " | ||
"support runtime introspection of type annotations. " | ||
"Only change it if you understand what that means. " |
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.
Could we explain "what that means" or link to the doc that explains it here ? :)
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.
My initial idea was something along the lines:
If you don't know what that means, you're probably better off to leave it as is.
I just wasn't sure this would be the right tone. It's never the less true. This should really only be set if the user fully understands the consequences. Not knowing runtime introspection
is a clear signal that he/she doesn't know enough.
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.
Sure, but this is the place where we should link to where you need to look if you want to try to understand, no ?
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.
@Pierre-Sassoulas I've updated the explanation. Please take a look and let me know what you think
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.
Nice, thank you for making this clearer !
Description
Clarify documentation for
typing
extension.Type of Changes
Related Issue
Closes #4545