Skip to content

Remove unused CommandLineParser package dependency#130244

Merged
jkotas merged 1 commit into
dotnet:mainfrom
akoeplinger:remove-commandlineparser
Jul 6, 2026
Merged

Remove unused CommandLineParser package dependency#130244
jkotas merged 1 commit into
dotnet:mainfrom
akoeplinger:remove-commandlineparser

Conversation

@akoeplinger

Copy link
Copy Markdown
Member

No description provided.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 CommandLineParser NuGet dependency from the test external package aggregation project and deletes its now-unused version property, reducing unnecessary package restore/copy surface in the test infrastructure.

Changes:

  • Removed CommandLineParser from src/tests/Common/external/external.csproj (PackageToInclude + PackageReference).
  • Removed CommandLineParserVersion from eng/Versions.props.
  • Verified there are no remaining references to CommandLineParser, CommandLineParserVersion, or common CommandLineParser-style usage patterns (e.g., using CommandLine;, [Verb], Reference Include="CommandLine").

Reviewed changes

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

File Description
src/tests/Common/external/external.csproj Drops the CommandLineParser package from the external test package set copied into targeting pack / CORE_ROOT.
eng/Versions.props Removes the obsolete CommandLineParserVersion property after the dependency removal.

@jkotas jkotas merged commit 96e71fe into dotnet:main Jul 6, 2026
170 of 174 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants