-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
remove deprecated items #3794
remove deprecated items #3794
Conversation
whirlpool, ripemd160, unsupportedextension, and the old interfaces
CHANGELOG.rst
Outdated
@@ -6,6 +6,9 @@ Changelog | |||
|
|||
.. note:: This version is not yet released and is under active development. | |||
|
|||
* **BACKWARDS INCOMPATIBLE:** ``Whirlpool``, ``RIPEMD160``, and | |||
``UnsupportedExtension`` have been removed. |
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.
MACContext
as well?
This should be clear they were already deprecated.
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.
We haven't removed MACContext
, just moved it. Not sure how to phrase that.
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.
Oh, right. Uhh, yeah, that's fine.
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.
The bit about making clear they were deprecated still stands.
There's also some unused imports, and |
I don't understand what's going on with docs on macOS? |
Ok, I wonder if this a segfault in pyenchant's wheels because x86 instructions or something. |
whirlpool, ripemd160, unsupportedextension, and the old interfaces
followup PR will remove RIPEMD160 and Whirlpool test vectors.
fixes #3793