Skip to content

Upgrade the .NET console app sample to .NET 8#1058

Merged
zhenlan merged 3 commits intomainfrom
upgrade
May 22, 2025
Merged

Upgrade the .NET console app sample to .NET 8#1058
zhenlan merged 3 commits intomainfrom
upgrade

Conversation

@zhenlan
Copy link
Member

@zhenlan zhenlan commented May 21, 2025

  1. Upgrade from .NET Core 3.1 to .NET 8
  2. Use the minimal project template style
  3. Switch to DefaultAzureCredential for authentication with Azure App Configuration

GitHub Copilot agent did most of the work :)

@zhenlan zhenlan requested a review from Copilot May 21, 2025 18:28
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

This PR upgrades the .NET console application sample from .NET Core 3.1 to .NET 8 using the minimal project template style and switches to using DefaultAzureCredential for authentication with Azure App Configuration.

  • Upgraded target framework and package versions in the project file.
  • Refactored the program to a simpler configuration setup and streamlined Azure App Configuration integration.
  • Updated the appsettings.json with a new property for the App Configuration endpoint.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
examples/DotNetCore/ConsoleApplication/appsettings.json Added AppConfigEndpoint placeholder and formatting adjustments
examples/DotNetCore/ConsoleApplication/Program.cs Refactored configuration setup, loop logic, and Azure connection
examples/DotNetCore/ConsoleApplication/ConsoleApplication.csproj Upgraded target framework and updated package references

@zhenlan zhenlan merged commit 269adf0 into main May 22, 2025
6 checks passed
@zhenlan zhenlan deleted the upgrade branch May 22, 2025 00:27
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