Skip to content

Initial PEP 792 types and index parsing#17311

Merged
woodruffw merged 10 commits intomainfrom
ww/pep792-types
Jan 7, 2026
Merged

Initial PEP 792 types and index parsing#17311
woodruffw merged 10 commits intomainfrom
ww/pep792-types

Conversation

@woodruffw
Copy link
Member

@woodruffw woodruffw commented Jan 2, 2026

Summary

Adds initial types and detail response handling for PEP 792 project status markers.

This is towards #15254.

Ref: https://packaging.python.org/en/latest/specifications/project-status-markers/#

Test Plan

Added new unit tests for both HTML and JSON index extraction.

@woodruffw woodruffw temporarily deployed to uv-test-registries January 2, 2026 22:39 — with GitHub Actions Inactive
@woodruffw woodruffw temporarily deployed to uv-test-registries January 3, 2026 05:18 — with GitHub Actions Inactive
@woodruffw woodruffw temporarily deployed to uv-test-registries January 4, 2026 04:57 — with GitHub Actions Inactive
#[derive(Debug, Clone)]
pub(crate) struct SimpleDetailHTML {
/// The PEP 792 project status information.
#[allow(dead_code)]
Copy link
Member Author

Choose a reason for hiding this comment

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

Flagging: this is currently marked as dead code, since nothing (outside tests) reads this field. My follow-on PRs will actually read this.

@woodruffw woodruffw temporarily deployed to uv-test-registries January 5, 2026 15:37 — with GitHub Actions Inactive
@woodruffw woodruffw temporarily deployed to uv-test-registries January 5, 2026 15:55 — with GitHub Actions Inactive
@woodruffw woodruffw temporarily deployed to uv-test-registries January 5, 2026 16:08 — with GitHub Actions Inactive
@woodruffw woodruffw marked this pull request as ready for review January 5, 2026 16:38
@woodruffw woodruffw mentioned this pull request Jan 5, 2026
6 tasks
@woodruffw woodruffw requested a review from konstin January 5, 2026 16:48
Copy link
Member

@konstin konstin left a comment

Choose a reason for hiding this comment

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

We should also add a test with a full simple JSON response, but I assume we'll do that anyway for the integration tests in the follow-ups.

Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
Signed-off-by: William Woodruff <william@astral.sh>
@woodruffw woodruffw enabled auto-merge (squash) January 7, 2026 16:25
@woodruffw woodruffw merged commit 9ddaae2 into main Jan 7, 2026
162 checks passed
@woodruffw woodruffw deleted the ww/pep792-types branch January 7, 2026 16:35
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.

2 participants