bpo-45150: Add hashlib.file_digest() for efficient file hashing by tiran · Pull Request #31930 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.7k
Merged
Conversation
bedevere-bot
added
the
awaiting core review
label
the-knights-who-say-ni
added
the
CLA signed
label
tiran
force-pushed
the
bpo-45150-file-digest
branch
from
1d6869b to
9b218da
Compare
tiran
force-pushed
the
bpo-45150-file-digest
branch
from
9b218da to
71f0401
Compare
tiran added 2 commits
tiran
requested a review
from gpshead
gpshead
requested changes
gpshead
left a comment
gpshead
left a comment
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like that you implemented this in pure Python.
Lib/hashlib.py Outdated
Lib/hashlib.py Outdated
Doc/library/hashlib.rst Outdated
bedevere-bot
added
awaiting changes
and removed
awaiting core review
labels
bedevere-bot
commented
Mar 16, 2022
bedevere-bot commented
When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.
gpshead
mentioned this pull request
tiran
requested a review
from gpshead
tiran
added
awaiting core review
and removed
awaiting changes
labels
gpshead
reviewed
Lib/test/test_hashlib.py Outdated
gpshead
approved these changes
bedevere-bot
added
awaiting merge
and removed
awaiting core review
labels
gpshead
reviewed
Lib/test/test_hashlib.py Outdated
tiran
requested a review
from gpshead
gpshead
approved these changes
tiran
added
the
🤖 automerge
label
miss-islington
merged commit
4f97d64
into
python:main
bedevere-bot
removed
the
awaiting merge
label
markshannon
commented
Mar 22, 2022
markshannon commented
Member
This may have broken the buildbots.
https://buildbot.python.org/all/#/builders/713/builds/452
tiran
commented
Mar 22, 2022
tiran commented
Member Author
This may have broken the buildbots. buildbot.python.org/all/#/builders/713/builds/452
FIPS ... let's see if GH-32046 solves the issue.
tiran
deleted the
bpo-45150-file-digest
branch
tiran
mentioned this pull request
tarekziade
mannequin
mentioned this pull request
This comment was marked as spam.
This comment was marked as spam.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment