search

dotnet / command-line-api

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

Value subsystem and core changes #2413

Closed KathleenDollard closed 1 month ago

KathleenDollard commented 2 months ago

Apologies for the big PR.

As I worked on the ValueSubsystem, I encountered unexpected issues in the core classes and had to fix them.

I also had a few things -scope changes and pipeline cleanup that I thought I could pull out and a conflict prohibited.

Looking forward to feedback.

KathleenDollard commented 2 months ago

Just changed to draft when I discovered whitespace issues, particularly in ParseResult. Need to walk dog, then will try to fix tonight.

KathleenDollard commented 2 months ago

I did a cleanup pass, but you will encounter some single line whitespace changes on lines with /* so that we will not continue to have these whitespace problems - particularly in ParseResult.