Skip to content
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

Use correct class to indicate present deprecation #8665

Merged
merged 2 commits into from Sep 22, 2022

Conversation

DavidCain
Copy link
Contributor

PendingDeprecationWarning means "we plan to deprecate, but haven't yet." A feature that's to be deleted in the next release is not planned to be deprecated; it is deprecated.

Base class for warnings about features which are obsolete and expected
to be deprecated in the future, but are not deprecated at the moment.

This class is rarely used as emitting a warning about a possible
upcoming deprecation is unusual, and DeprecationWarning is preferred for
already active deprecations.

(emphasis mine)

https://docs.python.org/3/library/exceptions.html#PendingDeprecationWarning

`PendingDeprecationWarning` means "we plan to deprecate, but haven't
yet." A feature that's to be deleted in the next release is not planned
to be deprecated; it **is** deprecated.

> Base class for warnings about features which are obsolete and expected
> to be deprecated in the future, but are not deprecated at the moment.
>
> This class is rarely used as emitting a warning about a possible
> upcoming deprecation is unusual, and DeprecationWarning is preferred for
> already active deprecations.

https://docs.python.org/3/library/exceptions.html#PendingDeprecationWarning
@kevin-brown kevin-brown merged commit 2de5081 into encode:master Sep 22, 2022
sigvef pushed a commit to sigvef/django-rest-framework that referenced this pull request Dec 3, 2022
`PendingDeprecationWarning` means "we plan to deprecate, but haven't
yet." A feature that's to be deleted in the next release is not planned
to be deprecated; it **is** deprecated.

> Base class for warnings about features which are obsolete and expected
> to be deprecated in the future, but are not deprecated at the moment.
>
> This class is rarely used as emitting a warning about a possible
> upcoming deprecation is unusual, and DeprecationWarning is preferred for
> already active deprecations.

https://docs.python.org/3/library/exceptions.html#PendingDeprecationWarning

Co-authored-by: Tom Christie <tom@tomchristie.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants