-
Notifications
You must be signed in to change notification settings - Fork 97
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
[v0.2] Deprecation warnings are ignored by default #27
Comments
There seems to be quite a long story behind it, see e.g. https://lwn.net/Articles/740804/ . I am in favour of activating them by default either with the |
As described in #29 (comment), we should advocate the use of |
Sounds good to me. It is probably sufficient to just include it in the docs and examples, similar to what is already done with |
I included sections on this in the documentaion as an example. Do you think this suffices? |
Yes, that looks good.
…On Thu, Aug 15 2019, euronion wrote:
I included sections on this in the documentaion as an example.
Do you think this suffices?
https://atlite.readthedocs.io/en/latest/examples/logfiles_and_messages.html
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#27 (comment)
|
Great. Closing the issue. |
Uh annoying, they worked wonders in Python 2.7. The default has been changed to ignore them in
3.2
unless you add-Wd
to your python interpreter or change the filter as you've shown. Hmm .. if I understand the manual correctly it's not intended for libraries to setcaptureWarnings
or change the filter. I suppose we should warn in the manual about that and change the examples accordingly.Do you think
logging.captureWarnings(True)
orwarnings.simplefilter('always', DeprecationWarning)
should be used? I'm tending towards the former.Originally posted by @coroa in #23
The text was updated successfully, but these errors were encountered: