-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Add runtime migration creation and application #37415
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
Open
thromel
wants to merge
49
commits into
dotnet:main
Choose a base branch
from
thromel:feature/runtime-migrations
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,195
−79
Open
Changes from 44 commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
633165e
Add runtime migration creation and application
thromel ed69364
Add CLI functional tests for CreateAndApplyMigration operation
thromel 872ffdc
Add SQLite integration tests for runtime migrations and fix service w…
thromel 2609bea
Add SQL Server integration tests for runtime migrations
thromel 52999ad
Fix CI build: Add DatabaseUpdateAddDescription to dotnet-ef Resources
thromel 62c8e9e
Fix lazy initialization of DynamicMigrationsAssembly
thromel 4a84f33
Trigger CI re-run
thromel 31ded81
Fix dry run test: remove query to non-existent database
thromel 9164772
Fix IsCI detection for Helix test environment
thromel 84eec3f
Add comprehensive tests for runtime migrations
thromel 4316282
Add RevertMigration support for runtime migrations
thromel 8e7cad7
Add comprehensive tests for runtime migrations
thromel 50dde25
Trigger CI re-run
thromel 234868e
Add migration name validation to CreateAndApplyMigration
thromel b78e506
Address PR review feedback for runtime migrations
thromel 848777b
Add missing using directive for Resources in DatabaseUpdateCommand
thromel 9303834
Address PR review feedback: remove IRuntimeMigrationService
thromel 953c86c
Add runtime migration tests to DesignTimeTestBase
thromel 9c8d771
Add missing runtime migration tests to DesignTimeTestBase
thromel b6a523b
Add comprehensive tests for runtime migrations feature
thromel 6f7ae79
Remove unrelated AddMigration tests from MigrationsOperationsTest
thromel f17dddc
Fix CI test failures for runtime migrations
thromel ac1deb1
Add rigorous schema verification tests for runtime migrations
thromel 6b0a93d
Add RuntimeMigrationSqlServerTest for CI compliance
thromel ad289d2
Fix SQLite file locking issue on Windows
thromel 13f5377
Address PR review comments for runtime migrations
thromel 3b99d66
Address remaining PR review comments for runtime migrations
thromel 4c48e09
Move Validate() to Configure.cs for dotnet-ef inclusion
thromel f0163c7
Fix SQL Server runtime migration test cleanup
thromel 01bfe2f
Fix connection state handling in CleanDatabase
thromel 355c7b7
Fix critical issues in runtime migrations
thromel 6e5e59f
Don't delete snapshot file on AddAndApplyMigration failure
thromel cd324fd
Address PR review comments for runtime migrations
thromel 5bf8a0b
Fix CI test failures
thromel 8129933
Fix remaining First() calls in Migration_preserves_existing_data test
thromel e21555c
Fix connection state in Migration_down_reverses_up test
thromel e9b3d54
Remove lock from AddMigrations per reviewer feedback
thromel a5e5331
Address reviewer feedback on test patterns
thromel e7b6329
Address remaining reviewer feedback
thromel 72a3c0f
Add createTables parameter to TestStore.CleanAsync for runtime migrat…
thromel a5b2659
Address Copilot review feedback
thromel 9a62a7b
Use existing MissingConditionalOption resource for --json validation
thromel 1d82f83
Fix GnomeContext to use valid SQLite connection string
thromel 6d64bb1
Fix GnomeContext to use externally opened SQLite connection
thromel 4193835
Use AssemblyTestContext in migration name tests
thromel 39498fa
Address runtime migrations review feedback
thromel f39d9a1
Align migration scaffolding and snapshot caching
thromel a2b3258
Allow null model snapshots in migrations assembly
thromel 489d35a
Handle null snapshots and save scaffolds centrally
thromel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.