Closed lchrusciel closed 3 years ago
Despite the note: Namespace is arbitrary but should be different from App\Migrations as migrations classes should NOT be autoloaded, the current Sylius Standard configuration suggests App\Migration as a namespace(ref. https://github.com/Sylius/Sylius-Standard/blob/master/config/packages/doctrine_migrations.yaml#L6). What is more, this namespace is not an issue anymore: https://github.com/doctrine/DoctrineMigrationsBundle/issues/242.
Namespace is arbitrary but should be different from App\Migrations as migrations classes should NOT be autoloaded
App\Migration
Without this change, current migration will not work automatically during plugin installation.
Thank you, Łukasz! :tada:
Despite the note:
Namespace is arbitrary but should be different from App\Migrations as migrations classes should NOT be autoloaded
, the current Sylius Standard configuration suggestsApp\Migration
as a namespace(ref. https://github.com/Sylius/Sylius-Standard/blob/master/config/packages/doctrine_migrations.yaml#L6). What is more, this namespace is not an issue anymore: https://github.com/doctrine/DoctrineMigrationsBundle/issues/242.Without this change, current migration will not work automatically during plugin installation.