migrations icon indicating copy to clipboard operation
migrations copied to clipboard

Fix up Migrations for Phinx 0.13 integration.

Open dereuromark opened this issue 4 years ago • 8 comments

Together with https://github.com/cakephp/phinx/pull/2033 this enables new Phinx major together with CakePHP 4.3+.

PHPStan passes, but Psalm has a few issues still it seems, even though they should be gone (I tried to clear cache, didn't help)

TODO: Also JSON part seems broken/problematic as per return type of parent.

dereuromark avatar Oct 15 '21 13:10 dereuromark

I might need a bit help for the remaining errors.

1) Migrations\Test\TestCase\Command\BakeMigrationDiffCommandTest::testEmptyHistoryNoMigrations
Symfony\Component\Console\Exception\InvalidArgumentException: The "configuration" option does not exist.

/home/runner/work/migrations/migrations/vendor/symfony/console/Input/Input.php:158
/home/runner/work/migrations/migrations/vendor/robmorgan/phinx/src/Phinx/Console/Command/AbstractCommand.php:239
/home/runner/work/migrations/migrations/vendor/robmorgan/phinx/src/Phinx/Console/Command/AbstractCommand.php:281
/home/runner/work/migrations/migrations/vendor/robmorgan/phinx/src/Phinx/Console/Command/AbstractCommand.php:117
/home/runner/work/migrations/migrations/src/Command/Phinx/CommandTrait.php:66
/home/runner/work/migrations/migrations/src/Command/Phinx/MarkMigrated.php:112

Any idea why this would be happening now?

dereuromark avatar Oct 15 '21 15:10 dereuromark

Not sure what the 0.next-assoc branch was in phinx? Should that be recreated?

MasterOdin avatar Nov 01 '21 17:11 MasterOdin

No, I updated those references, tests should now run.

dereuromark avatar Nov 01 '21 17:11 dereuromark

needs to rebase

saeideng avatar Nov 02 '21 13:11 saeideng

Yeah, wrong branch as well, I guess It should probably to towards 3.x now.

The 3.x release was bad, without those changes :/

dereuromark avatar Nov 02 '21 13:11 dereuromark

@dereuromark what is the status of this PR? Anything I can do to help this along to continue on releasing phinx 0.13?

MasterOdin avatar Nov 12 '21 19:11 MasterOdin

Do we have any blockers from Migrations side? I remember there were some tests failing that we didnt know how to handle.

dereuromark avatar Nov 12 '21 19:11 dereuromark

I suspect there are more test failures, but the merge conflicts in this PR need to be resolved first so that the test suite runs again? Also not sure if the test failures are things that need me to look at from the phinx side, or if they're more migrations specific stuff.

MasterOdin avatar Nov 12 '21 19:11 MasterOdin