mrahhal / Migrator.EF6

.NET Core CLI tool to enable EF6 migrations in an Asp.Net Core app.
MIT License
82 stars 15 forks source link

Rename Param ConnectionString to ConnectionStringForMigration #48

Closed xrkolovos closed 7 years ago

xrkolovos commented 7 years ago

Rename Param ConnectionString to ConnectionStringForMigration

mrahhal commented 7 years ago

Which ConnectionString do you mean?

hadiidbouk commented 7 years ago

@xrkolovos jewib = answer !!

xrkolovos commented 7 years ago

@hadiidbouk what do you mean?

public static string ConnectionString { get; set; }

mrahhal commented 7 years ago

@xrkolovos this is in the sample project. You can just name it whatever you want in your own DbContext.

mrahhal commented 7 years ago

But as an aside, ConnectionStringForMigration doesn't make much sense. This property will always be used by the DbContext. Its initial hard coded value is what gets used by the migrator.