-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Change deprecation warnings into errors for 7.0 release #8837
Conversation
This is a windows only dependency, which explains why it was missed until now.
43ad35d
to
95a1a1a
Compare
95a1a1a
to
9644f46
Compare
Oh, also, how does this work with warnings which weren't part of 6.3.0, but only introduced in this release? Looking at deprecated.py and the changelogs we seem to have many new deprecations which haven't been in a release yet? |
Bummer, yes you are right. Unfortunately we took so long to make a new release (with things landing on
I think 2) is the simplest one (much less work), and it just means some warnings will be visible longer, which I think is preferable than turning them into errors right away. In practice it just means closing this PR and scheduling the related issues to 8.0. @RonnyPfannschmidt @bluetech @asottile what do you guys think? |
I'd go with 2 -- no reason to revert them, can just not turn them into errors. |
I'd prefer 2 as well, especially because this major release is a bit special - it's only really a major release because of how much stuff accumulated. I'd rather keep 7.0 a rather painless release for people (reducing the pain to whatever will perhaps surface due to all our internal changes), and have the intended breakage in a 8.0 where we are a bit more flexible and hopefully not as pressured by not having had a release in a long time. |
Great, closing this then! I have updated the other issues as appropriate. 👍 |
Fix #7363
Close #8833