Skip to content

Conversation

@woodruffw
Copy link
Contributor

@woodruffw woodruffw commented Jan 30, 2024

Initial feedback from https://discuss.python.org/t/pep-740-index-support-for-digital-attestations/44498:

  • Adds a point with the JSON API version bump (consistent with PEP 700, which bumped the minor version for a backwards-compatible change)
  • Adds additional motivation/rationale context

📚 Documentation preview 📚: https://pep-previews--3637.org.readthedocs.build/

Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
@woodruffw woodruffw requested a review from dstufft as a code owner January 30, 2024 16:53
Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

Looks good from a PEP editor perspective

Copy link
Contributor

@sethmlarson sethmlarson left a comment

Choose a reason for hiding this comment

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

LGTM!

Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
@hugovk
Copy link
Member

hugovk commented Feb 10, 2024

@woodruffw Would you like to wait for sponsor @dstufft to review, or are you ready to merge?

@woodruffw
Copy link
Contributor Author

@woodruffw Would you like to wait for sponsor @dstufft to review, or are you ready to merge?

I'm ready for merge! I'm always happy to do additional PRs as needed, so I'd prefer to get this one in 🙂

@hugovk hugovk merged commit 708a729 into python:main Feb 10, 2024
@woodruffw woodruffw deleted the ww/740-feedback branch February 10, 2024 19:22
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