-
Notifications
You must be signed in to change notification settings - Fork 27.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
Hashlib specification #27038
Hashlib specification #27038
Conversation
@ArthurZucker, perhaps you can review this? |
Hey! As this might affect other HF repos we want to take our time and find a solution for all of them 🤗 gotta be aligned with other library maintainers! FYI @Wauplin |
That's fair. In case you need some references, similar changes have been included in other repos:
|
Thanks a lot @DueViktor for raising the question 🙏 I opened a PR in |
This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Please note that issues that do not follow the contributing guidelines are likely to be ignored. |
Closing this PR without merging. This has been handled in a separate PR (see #27034 for full context). |
What does this PR do?
The following PR address the issue raised in #27034.
Fixes # (issue)
Close #27034
Before submitting
Pull Request section?
to it if that's the case.
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.