Skip to content

feat(resolver): add VersionMapProvider for custom version resolution#981

Merged
mergify[bot] merged 1 commit intopython-wheel-build:mainfrom
rd4398:wrap-version-map
Mar 26, 2026
Merged

feat(resolver): add VersionMapProvider for custom version resolution#981
mergify[bot] merged 1 commit intopython-wheel-build:mainfrom
rd4398:wrap-version-map

Conversation

@rd4398
Copy link
Copy Markdown
Contributor

@rd4398 rd4398 commented Mar 25, 2026

Introduces VersionMapProvider class that wraps the VersionMap interface, allowing it to be used as a resolver provider in override hooks. This enables packages to define custom version-to-URL mappings that integrate seamlessly with the existing resolver infrastructure (PyPIProvider, GitHubTagProvider, etc.).

The provider supports constraints, caching, and follows the same patterns as other resolver providers, making it suitable for use in get_resolver_provider override hooks.

Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

Signed-off-by: Rohan Devasthale rdevasth@redhat.com

@rd4398 rd4398 requested a review from a team as a code owner March 25, 2026 19:05
@mergify mergify bot added the ci label Mar 25, 2026
@rd4398 rd4398 requested review from EmilienM and dhellmann March 25, 2026 19:06
@rd4398 rd4398 force-pushed the wrap-version-map branch from c63389f to 3088d60 Compare March 25, 2026 19:29
@rd4398
Copy link
Copy Markdown
Contributor Author

rd4398 commented Mar 25, 2026

I don't know why CI is failing 😞

@LalatenduMohanty
Copy link
Copy Markdown
Member

@rd4398 now that #983 has merged. We should rebase the PR against latest changes.

Introduces VersionMapProvider class that wraps the VersionMap interface,
allowing it to be used as a resolver provider in override hooks. This enables
packages to define custom version-to-URL mappings that integrate seamlessly
with the existing resolver infrastructure (PyPIProvider, GitHubTagProvider, etc.).

The provider supports constraints, caching, and follows the same patterns as
other resolver providers, making it suitable for use in get_resolver_provider
override hooks.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Rohan Devasthale <rdevasth@redhat.com>
@rd4398 rd4398 force-pushed the wrap-version-map branch from 3088d60 to a5fafd9 Compare March 26, 2026 13:15
@rd4398
Copy link
Copy Markdown
Contributor Author

rd4398 commented Mar 26, 2026

@rd4398 now that #983 has merged. We should rebase the PR against latest changes.

@LalatenduMohanty yes, done

Copy link
Copy Markdown
Member

@dhellmann dhellmann left a comment

Choose a reason for hiding this comment

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

One small suggestion for an improvement, but nothing to hold this up.

@rd4398 rd4398 mentioned this pull request Mar 26, 2026
@dhellmann
Copy link
Copy Markdown
Member

@Mergifyio refresh

@mergify mergify bot merged commit 0635c76 into python-wheel-build:main Mar 26, 2026
39 checks passed
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Mar 26, 2026

refresh

✅ Pull request refreshed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants