Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

PEP 740: tweak JSON simple API prescriptions #3768

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

woodruffw
Copy link
Contributor

@woodruffw woodruffw commented May 1, 2024

Per discussion with @dstufft: this removes the embedded provenance objects from the simple API and replaces them with digest references, much like the simple index. This has the virtuous effect of reducing the amount of mostly chaff JSON that client API consumers will need to download.

The added Appendix 3 has further details, including a rationale and concrete numbers. These have also been shared in the discussion thread.


馃摎 Documentation preview 馃摎: https://pep-previews--3768.org.readthedocs.build/

@woodruffw woodruffw requested a review from dstufft as a code owner May 1, 2024 19:19
peps/pep-0740.rst Outdated Show resolved Hide resolved
@woodruffw
Copy link
Contributor Author

Just leaving a comment here for myself: the PEP currently specifies that the "distribution name" goes into the attestation payload, but doesn't say anything about how that name is normalized. So we probably need some additional language in the PEP to say that sdist names get the PEP 625 treatment. Wheel names are already pre-normalized, although maybe we should also "ultranormalize" them to handle different postrelease spellings, etc.

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.

None yet

3 participants