Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds Azure OpenAI support to the WriteCommit tool, enabling users to choose between standard OpenAI and Azure OpenAI services during setup. The changes maintain backward compatibility while extending the tool's flexibility for enterprise users who prefer Azure's OpenAI implementation.
Key changes:
- Added Azure OpenAI SDK dependency and configuration support
- Enhanced setup wizard to allow choosing between OpenAI and Azure OpenAI
- Updated service layer to handle both OpenAI and Azure OpenAI clients
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| WriteCommit.csproj | Adds Azure.AI.OpenAI package dependency |
| Services/OpenAIService.cs | Implements Azure OpenAI client support with factory method |
| Services/ConfigurationService.cs | Extends setup wizard and configuration to support Azure OpenAI |
| README.md | Updates documentation to mention Azure OpenAI support |
| Program.cs | Updates CLI help text to reflect Azure OpenAI support |
| Models/AppConfiguration.cs | Adds UseAzureOpenAI configuration property |
Comments suppressed due to low confidence (1)
Services/ConfigurationService.cs:37
- The line has formatting issues with inconsistent spacing and alignment. Run
dotnet csharpier format Services/ConfigurationService.csto fix the formatting.
catch
| ChatClient testClient; | ||
| if (useAzure) | ||
| { | ||
| var azureClient = new Azure.AI.OpenAI.AzureOpenAIClient(new Uri(endpoint), new ApiKeyCredential(apiKey)); |
There was a problem hiding this comment.
The line has formatting issues with inconsistent spacing and alignment. Run dotnet csharpier format Services/ConfigurationService.cs to fix the formatting.
| { | ||
| if (_useAzure) | ||
| { | ||
| var azureClient = new Azure.AI.OpenAI.AzureOpenAIClient(new Uri(_endpoint), new ApiKeyCredential(_apiKey)); |
There was a problem hiding this comment.
The line has formatting issues with inconsistent spacing and alignment. Run dotnet csharpier format Services/OpenAIService.cs to fix the formatting.
1 participant
Summary
Azure.AI.OpenAIdependencyUseAzureOpenAIflag in configurationTesting
dotnet build WriteCommit.csprojdotnet test Write-Commit.slnhttps://chatgpt.com/codex/tasks/task_e_687a849a9208832a8784af6aa57b22c1