Skip to content

fix(resolver): delegate hook source providers - #1326

Merged
LalatenduMohanty merged 1 commit into
python-wheel-build:new-resolver-configfrom
LalatenduMohanty:fix/1324
Sep 11, 2026
Merged

fix(resolver): delegate hook source providers#1326
LalatenduMohanty merged 1 commit into
python-wheel-build:new-resolver-configfrom
LalatenduMohanty:fix/1324

Conversation

@LalatenduMohanty

@LalatenduMohanty LalatenduMohanty commented Sep 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Route hook-backed source resolver profiles through get_resolver_provider, forwarding resolver context and server URL while preserving legacy hook signatures. Validate returned providers and cover cooldown and error behavior.

Refs: #1324

Pull Request Description

What

Why

@LalatenduMohanty @codex
fix(resolver): delegate hook source providers
68c7d52 
Route hook-backed source resolver profiles through `get_resolver_provider`, forwarding resolver context and server URL while preserving legacy hook signatures. Validate returned providers and cover cooldown and error behavior.

Refs: python-wheel-build#1324
Co-Authored-By: Codex <noreply@openai.com>
Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
@LalatenduMohanty
LalatenduMohanty requested a review from a team as a code owner September 10, 2026 20:47
@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 732bc437-c914-49f8-8255-3ee68692667a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mergify mergify Bot added the ci label Sep 10, 2026
rd4398
rd4398 approved these changes Sep 11, 2026
View reviewed changes

@rd4398 rd4398 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@LalatenduMohanty
LalatenduMohanty merged commit 16fcfe4 into python-wheel-build:new-resolver-config Sep 11, 2026
37 checks passed
@mergify

mergify Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@LalatenduMohanty LalatenduMohanty mentioned this pull request Sep 11, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rd4398 rd4398 rd4398 approved these changes

Assignees

No one assigned

Labels

ci

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@LalatenduMohanty @rd4398