-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Be consistent about OnModelCreating as the terminology for fluent API #22343
Conversation
@Pilchie For RC1 |
6bcc088
to
4836b04
Compare
Approved for RC1 pending code review signoff if merged before 10am Pacific on 2020-09-01. |
4836b04
to
e33ea7f
Compare
Hello @AndriySvyryd! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
Will PR checks be done in next 38 mins? 😆 |
Removing approval and auto-merge. Please re-add For RC1, this will require Tactics approval, or M2 approval for RC2. |
macOS timed out again 😢 |
@Pilchie For RC2 |
Use "navigation" rather than "navigation property" Check usage of {foreignKey} and for correct quoting and use foreignKeyProperties where appropriate Fix AnnotationNotFound, BadFilterDerivedType, CircularDependency Update DisplayName for shared-type entity types to include the entity type name Part of #7201 Fixes #22310
e33ea7f
to
006a113
Compare
Approved for RC2. |
Use "navigation" rather than "navigation property"
Check usage of {foreignKey} and for correct quoting and use foreignKeyProperties where appropriate
Fix AnnotationNotFound, BadFilterDerivedType, CircularDependency
Update DisplayName for shared-type entity types to include the entity type name
Part of #7201
Fixes #22310