-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Support options batch for migrate:rollback.
#7531
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
Support options batch for migrate:rollback.
#7531
Conversation
|
@limingxinleo @huangdijia 再麻煩有空時 review 一下了 |
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.
Pull Request Overview
This PR adds batch option support to the migrate:rollback command, allowing users to rollback migrations by batch number instead of just by step count.
- Adds a new
--batchoption to the rollback command that accepts a batch number - Implements
getMigrationsByBatch()method in the migration repository to retrieve migrations by batch number - Updates rollback logic to handle batch-based rollbacks alongside existing step-based rollbacks
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| RollbackCommand.php | Adds batch option and passes it to migrator |
| Migrator.php | Implements batch rollback logic in getMigrationsForRollback method |
| MigrationRepositoryInterface.php | Adds getMigrationsByBatch method signature |
| DatabaseMigrationRepository.php | Implements getMigrationsByBatch method |
| DatabaseMigrationRollbackCommandTest.php | Updates tests to include batch parameter and adds batch-specific test |
| DatabaseMigrationRepositoryTest.php | Adds test for getMigrationsByBatch method |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
@huangdijia 上面的 review 調整完成了 |
batch for migrate:rollback.
|
节后回来发版本哈 |
No description provided.