-
-
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 8.0 #7363
Comments
@nicoddemus I want to work on this. Can you guide me a bit how to resolve this? |
@avats-dev thanks for the offer! However this is only done near the actual release (and actually this is incorrect, we do this in 7.1, I've updated the milestone), and we're nowhere near that. 😁 |
Ok 👍 |
It's a bit confusing that the original 7.0 milestone was apparently renamed to 8.0. In any case, this issue should either be renamed or remilestoned. |
@nicoddemus Is this something you're going to look into? If not, I think this needs some more context about what should be done. |
Sure, thanks for the ping, I will work on it. The idea is that we configure |
Currently RemovedIn8Warning is declared final, but really it should be sealed, because it means pytest can't create subclasses of RemovedIn8Warning, eg ReturnNotNoneWarning |
This was done in a4a189a |
Ref: #5584, #7362
Note: originally this was meant to be integrated in 7.0 in #8837, but we decided to postpone it as per #8837 (comment).
The text was updated successfully, but these errors were encountered: