Skip to content

[release/9.0] Don't use netcoreapp2.1 in dotnet-get-document#64351

Merged
wtgodbe merged 4 commits intorelease/9.0from
wtgodbe/GetDocument9
Nov 13, 2025
Merged

[release/9.0] Don't use netcoreapp2.1 in dotnet-get-document#64351
wtgodbe merged 4 commits intorelease/9.0from
wtgodbe/GetDocument9

Conversation

@wtgodbe
Copy link
Copy Markdown
Member

@wtgodbe wtgodbe commented Nov 13, 2025

netcoreapp2.1 is no longer supported

Copilot AI review requested due to automatic review settings November 13, 2025 18:23
@dotnet-policy-service dotnet-policy-service Bot added this to the 9.0.x milestone Nov 13, 2025
@wtgodbe wtgodbe added the tell-mode Indicates a PR which is being merged during tell-mode label Nov 13, 2025
@wtgodbe wtgodbe changed the title Don't use netcoreapp2.1 in dotnet-get-document [release/9.0] Don't use netcoreapp2.1 in dotnet-get-document Nov 13, 2025
Copy link
Copy Markdown
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 attempts to modernize the dotnet-getdocument tool by removing the legacy netcoreapp2.1 target framework. The tool now only targets net9.0 (via DefaultNetCoreTargetFramework).

Key changes:

  • Removed netcoreapp2.1 from the TargetFrameworks property in the project file
  • Simplified the tools directory resolution logic by removing version-specific conditionals

Reviewed Changes

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

File Description
src/Tools/dotnet-getdocument/src/dotnet-getdocument.csproj Removed netcoreapp2.1 from TargetFrameworks, now only targeting net9.0
src/Tools/dotnet-getdocument/src/Commands/InvokeCommand.cs Simplified tools directory resolution to use net{version} format for all .NET Core versions

Comment thread src/Tools/dotnet-getdocument/src/Commands/InvokeCommand.cs
This was referenced Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants