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

Powderhouse subsystems and directives #2351

Closed KathleenDollard closed 3 months ago

KathleenDollard commented 4 months ago

This addresses some of the issues raised in #2342 regarding directives (not response files environment variables(..

Remains in draft because:

KathleenDollard commented 3 months ago

This issue has files changed that I did not expect. I need to review this with Jean or Mikayla, regardless of content, and I know it makes it hard to review. So, unless you are in a patient mood, you can wait for a cleaner PR.

I did not deliberately delete most of the code marked as deleted.

KalleOlaviNiemitalo commented 3 months ago

The changes might be easier to understand if you updated the API compatibility test files too.

KathleenDollard commented 3 months ago

@KalleOlaviNiemitalo

The changes might be easier to understand if you updated the API compatibility test files too.

TBH, we are still getting everything to work with the split approach and have not given much thought to the API yet. We will do that a bit later.

I think the new subsystem approach will take some docs beyond API, and we're working on some design docs.