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

Make more internal code public #15452

Merged
merged 1 commit into from
Apr 23, 2019

Conversation

ajcvickers
Copy link
Contributor

@ajcvickers ajcvickers commented Apr 22, 2019

This is mostly adding proper docs to things that should be public but were still documented as internal. Also moved a few types accordingly and made a few things really internal or private.

Fixes #15442
Fixes #15441
Part of #15107

This is mostly adding proper docs to things that should be public but were still documented as internal. Also moved a few types accordingly and made a few things really internal or private.

Fixes #15442
Fixes #15441
Fixes #15107
@ajcvickers ajcvickers requested review from roji, maumar, AndriySvyryd, smitpatel and bricelam and removed request for maumar April 22, 2019 23:39
/// the same compatibility standards as public APIs. It may be changed or removed without notice in
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// Gets the transparent identifier type for the given inner and outer types.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for -> containing?

Copy link
Contributor

@maumar maumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

query stuff looks good

Copy link
Contributor

@bricelam bricelam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Migrations changes look good

@ajcvickers ajcvickers merged commit 2e5a432 into master Apr 23, 2019
@smitpatel smitpatel deleted the InsideInformalOutsideStormyInsideNormalAgain0422 branch April 30, 2019 17:46
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.

Make RelationalCompiledQueryCacheKeyGenerator really public Make ICurrentDbContext public
4 participants