dotnet / command-line-api

Command line parsing, invocation, and rendering of terminal output.
https://github.com/dotnet/command-line-api/wiki
MIT License
3.35k stars 375 forks source link

Support for ValueResult as public type to carry value and location #2372

Closed KathleenDollard closed 2 months ago

KathleenDollard commented 3 months ago

I am having an issue where the base of this merge is not what I am expecting. You can see the problem in System.CommandLine.Subsystems/Directives/DirectiveSubsystem.cs which exists in dotnet/command-line-api, but appears as completely fresh code in this PR. Another example is that ParserTests.cs has a bunch of whitespace fixes made by @mhutch that do not appear in the base of this PR.

I either did something dumb or I have lost my mind.

Commits in this PR also need to be cleaned up, so it is not ready to go, but I wanted to put this out so I could get help resolving this.