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

#5737 Expose Current ConventionSet #5741

Closed
wants to merge 1 commit into from
Closed

#5737 Expose Current ConventionSet #5741

wants to merge 1 commit into from

Conversation

rjperes
Copy link

@rjperes rjperes commented Jun 13, 2016

Simple change, just expose the private ConventionSet so that we can add to the current conventions.
Please see #5737 and https://weblogs.asp.net/ricardoperes/implementing-missing-features-in-entity-framework-core-part-4-conventions for a description of the problem.

@dnfclas
Copy link

dnfclas commented Jun 13, 2016

Hi @rjperes, 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;

@rjperes
Copy link
Author

rjperes commented Jun 13, 2016

Weird... The only change was the addition of a property... if the tests are failing, then they are already failing in the source branch, from which I forked!

@smitpatel
Copy link
Contributor

@rjperes - Tests are failing because the property which is added needs to be virtual.

@AndriySvyryd
Copy link
Member

@rjperes Thanks for your contribution. Unfortunately it goes against the design we are pursuing. Please see #5737 (comment)

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.

4 participants