feat(resolver): dispatch configured source resolvers - #1325
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
b7c3c24 to
9cdcb36
Compare
Route packages with source resolver configuration through the configured provider while preserving legacy override behavior. Widen `AbstractResolver.resolver_provider` to accept `RequirementType | None` and remove the `typing.cast` that hid the mismatch. Add coverage for source and bootstrap resolution paths and document the dispatch. Refs: python-wheel-build#1292 Co-Authored-By: Claude <claude@anthropic.com> Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
…lvers Source resolvers bypassed `resolve_package_cooldown()`, so the global `--min-release-age` flag was silently ignored and top-level `==` pins did not disable the cooldown. Extract `_effective_cooldown()` from `resolve_package_cooldown()` and call it in the source-resolver dispatch path so both legacy and configured resolvers share the same cooldown merge logic. Refs: python-wheel-build#1292 Co-Authored-By: Claude <claude@anthropic.com> Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
9cdcb36 to
b3de3e3
Compare
|
Tick the box to add this pull request to the merge queue (same as
|
a8984bc
into
python-wheel-build:new-resolver-config
2 participants
Route packages with source resolver configuration through the configured provider while preserving legacy override behavior. Add coverage for source and bootstrap resolution paths and document the dispatch.
Refs: #1292
Pull Request Description
What
Why