Refactor Azure client initialization to reduce code duplication #823
+63
−92
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes being requested
This PR streamlines the initialization process for AzureOpenAI and AsyncAzureOpenAI clients by centralizing common configuration steps. The main change involves the introduction of a
_configure_client_settings
method inBaseAzureClient
, which handles the setup of API keys, tokens, and endpoints.Key Changes:
BaseAzureClient
.AzureOpenAI
andAsyncAzureOpenAI
to use the new configuration method.Benefits:
This update enhances the code's clarity and maintainability without introducing breaking changes.
Additional context & links