-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Deprecation warning - alternative #3615
Conversation
Updated to now also print warnings for both but hides the known ones for non-debug. |
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.
Few things that should be changed if we choose to handle #3612 in this way.
Also, this seems to missing one of the warnings mentioned in #3529?
<frozen importlib._bootstrap>:219: DeprecationWarning: an integer is required (got type float). Implicit conversion to integers using __int__ is deprecated, and may be removed in a future version of Python.
I have also updated #3529 to make it easier to track these changes. |
@jack1142 I can remove the error messages and put a clearer error message if you want to reduce the clutter. |
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.
LGTM
Alternative to #3613
Hides the known ones for non-debug and shows all for debug.
I did line number variant but could also do regex but I think line number may be better in this case incase there is two errors with same error but different linenumber. This could also be checked easily if it's fixed by debug.