-
Notifications
You must be signed in to change notification settings - Fork 850
Remove Microsoft.Extensions.AI.AzureAIInference #7096
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Azure.AI.Inference is no longer recommended. We don't need to maintain the source for the wrapper.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes the Microsoft.Extensions.AI.AzureAIInference library and all its related code, following Microsoft's recommendation that Azure.AI.Inference is no longer the recommended approach. The library has not been actively developed since there have been no changes in the underlying Azure.AI.Inference library to react to.
Key Changes:
- Complete removal of the AzureAIInference integration library source code
- Removal of all associated test files and test infrastructure
- Removal of documentation (README.md, CHANGELOG.md)
- Removal of the Azure.AI.Inference package dependency from the repository
Reviewed changes
Copilot reviewed 18 out of 19 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/Libraries/Microsoft.Extensions.AI.AzureAIInference/* |
Removed entire source library including chat client, embedding generators, extensions, and supporting types |
test/Libraries/Microsoft.Extensions.AI.AzureAIInference.Tests/* |
Removed all test files including unit tests, integration tests, and test helpers |
eng/packages/General.props |
Removed Azure.AI.Inference package reference from version management |
The PR is clean and comprehensive - it removes all traces of the AzureAIInference integration without leaving orphaned references. All deleted files are properly removed as complete files (entire content deleted), which is the appropriate approach for deprecating an entire library.
|
From a global search: extensions/test/Libraries/Microsoft.Extensions.AI.Integration.Tests/README.md Lines 44 to 59 in 40d04a7
Line 19 in 40d04a7
|
Co-authored-by: jozkee <[email protected]>
* Initial plan * Remove AzureAIInference remnants after PR #7096 Co-authored-by: jozkee <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: jozkee <[email protected]>
Azure.AI.Inference is no longer recommended, e.g. see the "Important" notice at https://learn.microsoft.com/en-us/azure/ai-foundry/foundry-models/supported-languages. We haven't been making any changes to the wrapper, since there hasn't been any changes in the underlying library to react to, and we don't need to continue maintaining the source for the wrapper.
Microsoft Reviewers: Open in CodeFlow