Skip to content
This repository has been archived by the owner on Dec 12, 2020. It is now read-only.

State of System.CommandLine #63

Closed
LokiMidgard opened this issue Mar 31, 2018 · 2 comments
Closed

State of System.CommandLine #63

LokiMidgard opened this issue Mar 31, 2018 · 2 comments

Comments

@LokiMidgard
Copy link
Contributor

Are there any plans to change something on how the arguments are parsed.

If I see it correctly, the only reason the consumer of a generator written with this framework needs to put a nuget.config file in his project is because of how the arguments are parsed.

Looking at the repository, it does not seems like it will be out form beta (and on nuget) anytime soon.

I would like to have as less friction for the consumers of my libs as possible. Best case would be that only a nuget install is needed.

In my opinion the nuget.config could easily be avoided.

  • Forking System.CommandLine and using the fork (until the original is out on nuget)
  • Using a different command line parser (don't know what's out there)
  • writing an own parser

I think the first one would be a valid approach. Any thoughts on why this is a dumb idea?

@AArnott
Copy link
Owner

AArnott commented Apr 1, 2018

Not a bad idea. I'll check out options and keep this issue active to track.

@amis92
Copy link
Collaborator

amis92 commented May 7, 2018

It's officially archived: dotnet/corefxlab#2245

So it'll never move out of MyGet. It's the final sign that this repo need to either copy the code over (it's under MIT so no prob) or just use another library completely.

AArnott added a commit that referenced this issue May 10, 2018
Inline System.CommandLine into Tool project, closes #63
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants