Skip to content

6.0 API Reviews #24743

Closed
Closed

Description

February 19

Done
  • Investigate removing IModelCreationDependencies
  • Rename SingletonModelDependencies to RuntimeModelDependecies
  • Consider renaming IConventionModel.GetEntityTypes(Type type); to FindEntityTypes
  • Rename IConventionModel.FindPrincipals to GetPrincipals
  • @AndriySvyryd Add RelationalEntityTypeExtensions.FindRowInternalForeignKeys IEntityType overload, and more generally have three overloads for each of these.
  • @AndriySvyryd RelationalPropertyExtensions.FindOverrides may not need a mutable overload

March 4

Done
  • Change Annotatable.IsReadonly to ReadOnly
  • Change EnsureReadonly to EnsureMutable
  • @ajcvickers Review template methods on RelationalConnection and remove DbConnectionState

March 24

Done
  • Rename DisableConcurrencyDetection to EnableThreadSafetyChecks
  • @bricelam Check that dropping ITableMigrationOperation from DropIndexOperation is expected.
  • Rename IRelationalCommand.PopulateFromTemplate to PopulateFrom
  • Rename parameter of above from templateCommand to command

April 21

Done
  • Move DbContext.DesignTimeModel to GetService<IDesignTimeModel>() or similar
  • @AndriySvyryd Consider making ForeignKey.ReferencingSkipNavigations return an ISet
  • @AndriySvyryd Consider if SlimModel.RelationalModel can be made an explicit interface implementation and will still show up in the debugger. Also consider for DebugView.

June 16

Done
  • @AndriySvyryd Make CSharpRuntimeAnnotationCodeGenerator and friends internal so we can add language-agnostic APIs later (Make ICSharpRuntimeAnnotationCodeGenerator language-agnostic #25110), try to move ICSharpHelper back to .Design
  • @AndriySvyryd Make UpdateEntryEventData derive from DbContextEventData
  • @AndriySvyryd Move more classes under the Microsoft.EntityFrameworkCore.Design namespace in Relational into the core assembly
  • @AndriySvyryd Make RelationalCSharpRuntimeAnnotationCodeGenerator pubternal
  • @ajcvickers Why did ContextParamaterBinding change to an array of IPropertyBase?
    • Should it have params?
  • @ajcvickers Review IRelationalCommandDiagnosticsLogger
  • @smitpatel Remove IClonable from FromSqlExpression and TableExpression. Pattern match or introduce a new interface.
    • Clone should return something more specific than object
  • @smitpatel Mitigate future breaks to SelectExpression. Mark more members as pubternal.

August 2

Done
  • @roji Why is CoreOptionsExtension.DefaultLoggingCacheTime public? Make DefaultLoggingCacheTime private #25422
  • @AndriySvyryd Rename IMutableModificationCommandFactory to just IModificationCommandFactory
  • @AndriySvyryd Make ColumnModificationParameters and ModificationCommandParameters structs and use in
  • @AndriySvyryd Rename IModificationCommand to IReadOnlyModificationCommand
    • Rename IMutableModificationCommand to IModificationCommand
  • @ajcvickers Move commandSource constructor parameters to the end in *EventData classes and IRelationalCommandDiagnosticsLogger
  • @ajcvickers Address confusion beween AddSqlite and AddEntityFrameworkSqlite
  • @maumar Make IQueryRootCreator a generic extensitiliby helper for nav expansion
  • @maumar Can we mitigate timezone issues on TemporalAll/AsOf/etc methods?
  • @maumar Rename With to Use in new methods on SqlServerEntityTypeBuilderExtensions
  • @maumar Remove Temporal from new methods in SqlServerEntityTypeExtensions
    • Keep it in the underlying annotation names
  • @maumar Add Core and Relational dependency objects to SqlServerTemporalConvention
  • @bricelam Move MigrationsBundle to the Migrations namespace
    • Rename parameter to contextTypeName
  • @bricelam Host API Review of CLI and PMC commands

August 18

  • @bricelam Rename --bundle-runtime to --target-runtime
    • Rename -Runtime to -TargetRuntime in PMC
  • @bricelam Remove --bundle-configuration
  • @bricelam Rename default bundle name to efbundle
  • @AndriySvyryd Rename Scalars to DefaultTypeMapping
    • Singularize methods in ScalarConfigurationBuilder and ScalarConfigurationBuilderExtensions
  • @ajcvickers Remove CosmosDbContextOptionsBuilder.HttpClientFactory and enable ReplaceService instead. Looked at this and talked to Andriy. It's quite complicated and somewhat ugly to do a service, so leaving as is.
  • @smitpatel Can we remove the override of TranslateSubquery in RelationalQueryableMethodTranslatingExpressionVisitor?
  • @ajcvickers Revert ProviderCodeGenerator.RelationalDependencies to just Dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

area-globalclosed-fixedThe issue has been fixed and is/will be included in the release indicated by the issue milestone.type-enhancement

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions