Skip to content

Conversation

@tmat
Copy link
Member

@tmat tmat commented Jan 9, 2026

Adds two new projects:

  • Microsoft.DotNet.Cli.CoreUtils: basic utilities with minimal dependencies (unlike Microsoft.DotNet.Cli.Cli.Utils)
  • Microsoft.DotNet.Cli.Definitions: contains command definitions
    Dependencies:
    • Microsoft.DotNet.Cli.CoreUtils
    • Microsoft.DotNet.Cli.CommandLine
    • NuGet.Versioning

Moves all command definitions, except for new and nuget, to Microsoft.DotNet.Cli.Definitions.

Use Newtonsoft.Json package instead of Humanizer for testing to simplify dependencies.
Restore of Humanizer sometimes fails with:

Determining projects to restore...
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : warning NU1507: There are 13 package sources defined in your configuration. When using central package management, please map your package sources with package source mapping (https://aka.ms/nuget-package-source-mapping) or specify a single package source. The following sources are defined: darc-pub-dotnet-dotnet-fad253f, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, dotnet10, dotnet10-transport, dotnet-public, dotnet-tools, test-tools
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.bn. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.ca. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.fi. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.fil. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.ko. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.lb. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.lt. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.ms. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.pt-BR. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages
C:\h\w\A4A70936\t\dotnetSdkTests\di2id515.eqw\FileBasedApp_---36A7B5DD_5\Program.csproj : error NU1101: Unable to find package Humanizer.Core.th. No packages exist with this id in source(s): darc-pub-dotnet-dotnet-fad253f, dotnet-public, dotnet-tools, dotnet10, dotnet10-transport, dotnet6, dotnet7, dotnet8, dotnet8-transport, dotnet8-workloads, dotnet9, dotnet9-transport, Package source 1, test-tools, testpackages

Follow up:

  • Move new and nuget
  • Clean up duplicate/unused resource strings

@tmat tmat marked this pull request as ready for review January 9, 2026 21:02
@tmat tmat requested review from a team as code owners January 9, 2026 21:02
@tmat
Copy link
Member Author

tmat commented Jan 9, 2026

@MiYanni ptal

Copy link
Member

@MiYanni MiYanni left a comment

Choose a reason for hiding this comment

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

Overall changes look good. Just a few things I commented on to take a look at.

@tmat
Copy link
Member Author

tmat commented Jan 11, 2026

/ba-g ssl

@tmat
Copy link
Member Author

tmat commented Jan 11, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tmat tmat force-pushed the DefinitionsProject branch 2 times, most recently from cc6afc9 to 76998cb Compare January 14, 2026 20:41
@tmat tmat changed the base branch from release/10.0.2xx to release/10.0.3xx January 16, 2026 16:27
@tmat tmat enabled auto-merge (squash) January 16, 2026 16:27
@mmitche mmitche deleted the branch dotnet:release/10.0.3xx January 17, 2026 00:40
@mmitche mmitche closed this Jan 17, 2026
auto-merge was automatically disabled January 17, 2026 00:40

Pull request was closed

@mmitche mmitche reopened this Jan 17, 2026
@mmitche mmitche requested review from a team as code owners January 17, 2026 00:51
@mmitche mmitche requested review from a team and phil-allen-msft as code owners January 17, 2026 00:51
@tmat tmat marked this pull request as draft January 18, 2026 17:22
@tmat tmat changed the base branch from release/10.0.3xx to release/10.0.2xx January 18, 2026 17:22
@tmat tmat changed the title Separate command definitions to a new project [do not merge] Separate command definitions to a new project Jan 20, 2026
@tmat tmat marked this pull request as ready for review January 20, 2026 20:56
@tmat tmat force-pushed the DefinitionsProject branch from 2155165 to b041146 Compare January 20, 2026 21:15
@tmat tmat force-pushed the DefinitionsProject branch from b041146 to a6572b5 Compare January 20, 2026 21:15
@tmat tmat changed the base branch from release/10.0.2xx to release/10.0.3xx January 20, 2026 21:15
@tmat tmat changed the title [do not merge] Separate command definitions to a new project Separate command definitions to a new project Jan 20, 2026
@tmat tmat merged commit 6cadd91 into dotnet:release/10.0.3xx Jan 21, 2026
26 checks passed
@tmat tmat deleted the DefinitionsProject branch January 21, 2026 01:31
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