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
Resolve -From arguments in Script-Migration [WRONG BRANCH] #6279
Conversation
…ameworkCore.Relational.Tests This is required to build the package when referencing the test specification as a package (instead of as a src)
# Conflicts: # test/Microsoft.EntityFrameworkCore.FunctionalTests/project.json # test/Microsoft.EntityFrameworkCore.InMemory.FunctionalTests/project.json # test/Microsoft.EntityFrameworkCore.Microbenchmarks.EF6/project.json # test/Microsoft.EntityFrameworkCore.Microbenchmarks/project.json # test/Microsoft.EntityFrameworkCore.SqlServer.Design.FunctionalTests/project.json # test/Microsoft.EntityFrameworkCore.SqlServer.FunctionalTests/project.json # test/Microsoft.EntityFrameworkCore.Sqlite.FunctionalTests/project.json # test/Microsoft.EntityFrameworkCore.Tests/project.json
Nit: probably time to rebase. GH is picking up commits not related to the actual PR |
@"ALTER TABLE ""Table1"" RENAME TO ""Table2""; | ||
|
||
INSERT INTO ""__EFMigrationsHistory"" (""MigrationId"", ""ProductVersion"") | ||
VALUES ('00000000000002_Migration2', '7.0.0-test'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: EF 7? What's that?
LGTM. |
Closing. My merges into branch got renamed and it's messed up the PR. |
If only we had known this was coming ;) https://github.com/blog/2224-change-the-base-branch-of-a-pull-request |
Ha! About time... |
Fixes #6126 & fixes #6228