Skip to content

Conversation

@linglingye001
Copy link
Member

No description provided.

@linglingye001 linglingye001 requested review from Copilot and zhenlan July 14, 2025 08:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Add a Go console chat application sample that integrates Azure App Configuration for dynamic AI settings and Azure OpenAI for chat completions.

  • Introduce main.go with interactive chat logic, configuration loading, and OpenAI client setup
  • Provide README.md documenting setup, usage, and troubleshooting

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
examples/Go/ChatApp/main.go Added interactive chat app, Azure App Configuration loading, and Azure OpenAI client integration
examples/Go/ChatApp/README.md Added documentation for sample prerequisites, configuration keys, usage, and troubleshooting
Comments suppressed due to low confidence (2)

examples/Go/ChatApp/README.md:73

  • [nitpick] The sample output shows "Exiting chat. Goodbye!", but in code it prints "Exiting Chat. Goodbye!" (capital C). Consider aligning the case in the README with the actual behavior.
Exiting chat. Goodbye!

examples/Go/ChatApp/README.md:36

  • [nitpick] The README uses ApiKey, while the struct field is APIKey. Ensure the key name matches exactly or note how the provider handles case-insensitive mapping.
- `ChatApp:AzureOpenAI:ApiKey` - Key Vault reference to the API key for Azure OpenAI (optional)

@linglingye001 linglingye001 merged commit 1c74a30 into Azure:main Sep 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants