Feat/1293 download dispatch - #1338
LalatenduMohanty wants to merge 1 commit into
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: Repository: python-wheel-build/fromager/.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 |
b777a35 to
73a31b2
Compare
6a75732 to
ccc670f
Compare
When a package has a `source:` resolver configured, delegate `download_source()` to `source_resolver.download()` instead of the legacy override-hook path. Change the return type from `Path` to `(Path, DownloadKind)` so callers know what artifact was downloaded (sdist, tarball, prebuilt wheel, or git checkout). Also implement `resolver_provider()` for the `hook-sdist` and `hook-prebuilt` resolver profiles, which invoke the package's `get_resolver_provider` override hook and validate the result. Closes: python-wheel-build#1293 Co-Authored-By: Claude <claude@anthropic.com> Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
ccc670f to
ef28435
Compare
This branch has not been deployed
1 participant
Pull Request Description
Route configured source downloads through their resolver so the download boundary exposes artifact kinds to later phases while preserving legacy package behavior.
Closes #1293