Thanks to visit codestin.com
Credit goes to github.com

Skip to content

PEP 710: elaborate on storing at least one hash #3884

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fridex
Copy link
Contributor

@fridex fridex commented Aug 1, 2024


πŸ“š Documentation preview πŸ“š: https://pep-previews--3884.org.readthedocs.build/

@fridex fridex requested a review from dstufft as a code owner August 1, 2024 06:13
@fridex
Copy link
Contributor Author

fridex commented Aug 1, 2024

CC @charliermarsh @zanieb @sethmlarson as per discussion on Discourse, please let us know if you are fine with this

Comment on lines +660 to +685
Thanks to Gregory P. Smith, StΓ©phane Bidoul, C.A.M. Gerlach, and Adam Turner
for reviewing this PEP and providing valuable suggestions.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Too kind!

@charliermarsh
Copy link

I'm fine with it, thank you.

@fridex fridex force-pushed the pep-710-uv-no-hash branch 2 times, most recently from 6cb768f to f396d89 Compare August 2, 2024 09:43
Copy link
Member

@AA-Turner AA-Turner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pending one comment, looks good

@fridex fridex force-pushed the pep-710-uv-no-hash branch from f396d89 to 89a5c0a Compare June 26, 2025 13:05
@fridex fridex force-pushed the pep-710-uv-no-hash branch from 89a5c0a to 1b7f62d Compare June 26, 2025 13:05
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.

4 participants