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

There should be a ConfigurationProvider #104

Open
MovGP0 opened this issue Dec 17, 2018 · 2 comments
Open

There should be a ConfigurationProvider #104

MovGP0 opened this issue Dec 17, 2018 · 2 comments

Comments

@MovGP0
Copy link

MovGP0 commented Dec 17, 2018

There should be a reference implementation of Microsoft.Extensions.Configuration.IConfigurationProvider that works with the FluentCommandLineParser class.

https://andrewlock.net/creating-a-custom-iconfigurationprovider-in-asp-net-core-to-parse-yaml/

@siywilliams
Copy link
Member

Looks interesting but I've not come across IConfigurationProvider myself yet.

My concern is adding a dependency to Microsoft.Extensions.Configuration (FCLP currently has no external dependencies, something I've tried to maintain).

The correct way to go about this would be to create another extension project, for example something like FluentCommandLineParser.ConfigurationProvider which would provide the extensions to add this functionality without polluting the main project.

@astrohart
Copy link

@siywilliams that is the way to go. Separate extension project and in its own NuGet package. This way, I can add that to my project only if I want/need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants