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

Adding a Json provider in the same vein as the Xml provider. #465

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pagan-gerbil
Copy link

Copy of Xml package reworked to read JSON files. Branched off of last nuget release to help this get published quickly if possible; only a minor change needed in JsonFileValuesProvider to make it compatible with the main branch HEAD.

Still want to add unit tests and an appropriate sample/documentation - if this is accepted I can justify the time to work on those.

@paillave
Copy link
Owner

Hello, thanks a lot for the PR. Could you make it against develop instead?

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

Successfully merging this pull request may close these issues.

2 participants