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

Partial update to ASP.NET Core RC2 #1250

Closed
wants to merge 1 commit into from
Closed

Conversation

kdaveid
Copy link

@kdaveid kdaveid commented May 19, 2016

Usage of MemoryConfigurationSource and shifting to Microsoft.Extensions.Configuration and Microsoft.Extensions.Configuration.Memory as MemoryConfigurationProvider needs an MemoryConfigurationSource.

https://github.com/aspnet/Configuration/blob/dev/src/Microsoft.Extensions.Configuration/MemoryConfigurationProvider.cs

Usage of MemoryConfigurationSource and shifting to Microsoft.Extensions.Configuration and Microsoft.Extensions.Configuration.Memory as MemoryConfigurationProvider needs an MemoryConfigurationSource.

https://github.com/aspnet/Configuration/blob/dev/src/Microsoft.Extensions.Configuration/MemoryConfigurationProvider.cs
@dnfclas
Copy link

dnfclas commented May 19, 2016

Hi @kdaveid, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@kdaveid
Copy link
Author

kdaveid commented May 19, 2016

Didnt' see c304161 so I drop this PR.

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.

None yet

2 participants