Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace parser with System.CommandLine #201

Closed
natemcmaster opened this issue Dec 19, 2018 · 3 comments
Closed

Replace parser with System.CommandLine #201

natemcmaster opened this issue Dec 19, 2018 · 3 comments
Labels
breaking-change closed-wontfix This issue is closed because there are no plans to fix this. needs-design

Comments

@natemcmaster
Copy link
Owner

The .NET team is working on creating a parsing library tentatively called System.CommandLine. See https://github.com/dotnet/command-line-api.

I am considering replacing the parser in this library with System.CommandLine. It would be a substantial breaking change, but this parser has a much richer API for parsing along with some other nice features that I don't want to re-invent on my own.

This change needs more investigation.

@atruskie
Copy link
Contributor

FWIW part of the reason i chose this library was because it was an adaptation of System.CommandLine. I was hoping if they finished work in System.CommandLine then the delta between my app and the standard impl would be smaller. So 👍

@TheConstructor
Copy link
Contributor

TheConstructor commented Jan 1, 2019

So I guess this depends in part on dotnet/command-line-api#356. What will this project become? Configuration-infrastructure (I believe they currently don't have annotation support) and helpers?

I know it says needs-design and breaking-change, but I am really interested how deep and how far you plan this to go.

@natemcmaster natemcmaster added the closed-wontfix This issue is closed because there are no plans to fix this. label Jan 6, 2019
@natemcmaster
Copy link
Owner Author

I gave this some more thought over the last few weeks. At this point, I'm not planning to make this change. (see also #206 for my thoughts on what will become of this project)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change closed-wontfix This issue is closed because there are no plans to fix this. needs-design
Projects
None yet
Development

No branches or pull requests

3 participants