Skip to content

Conversation

@rajkumar-rangaraj
Copy link
Member

@rajkumar-rangaraj rajkumar-rangaraj commented Jan 28, 2026

Removes netstandard2.0 from the following packages:

  • Microsoft.ApplicationInsights
  • Microsoft.ApplicationInsights.AspNetCore
  • Microsoft.ApplicationInsights.WorkerService

These packages now target net8.0, net9.0, net10.0 (and net462 for the core SDK).

Copilot AI review requested due to automatic review settings January 28, 2026 02:34
@rajkumar-rangaraj rajkumar-rangaraj changed the title Rajrang/rem netstandard Remove netstandard2.0 target framework from core packages Jan 28, 2026
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 removes the netstandard2.0 target framework from three Application Insights packages: Microsoft.ApplicationInsights, Microsoft.ApplicationInsights.AspNetCore, and Microsoft.ApplicationInsights.WorkerService. This simplifies the build targets and indicates these packages will no longer support .NET Standard 2.0, requiring users to target .NET 8.0 or higher (or .NET Framework 4.6.2 for the base package).

Changes:

  • Removed netstandard2.0 from target frameworks in three project files
  • Cleaned up conditional compilation directives and build configurations that referenced netstandard2.0
  • Updated CHANGELOG.md to document the removal

Reviewed changes

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

File Description
NETCORE/src/Microsoft.ApplicationInsights.WorkerService/Microsoft.ApplicationInsights.WorkerService.csproj Removed netstandard2.0 from TargetFrameworks, now targets only net10.0, net9.0, and net8.0
NETCORE/src/Microsoft.ApplicationInsights.AspNetCore/Microsoft.ApplicationInsights.AspNetCore.csproj Removed netstandard2.0 from TargetFrameworks and removed the conditional TargetFrameworks for non-Windows OS, also removed the Release configuration PropertyGroup specific to netstandard2.0
BASE/src/Microsoft.ApplicationInsights/Microsoft.ApplicationInsights.csproj Removed netstandard2.0 from both Windows and non-Windows TargetFrameworks, removed commented-out PackageReference, and updated ItemGroup condition to exclude only net8.0/9.0/10.0
CHANGELOG.md Added entry documenting the removal of netstandard2.0 target framework from the three packages

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rajkumar-rangaraj rajkumar-rangaraj merged commit 2797c90 into main Jan 28, 2026
18 checks passed
@rajkumar-rangaraj rajkumar-rangaraj deleted the rajrang/RemNetstandard branch January 28, 2026 21:30
This was referenced Feb 1, 2026
This was referenced Feb 9, 2026
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.

2 participants