-
Notifications
You must be signed in to change notification settings - Fork 71
Implementando migrations #76
base: master
Are you sure you want to change the base?
Conversation
Criei um PR para que possamos discutir sobre a migrations. Se estiverem de acordo vou melhorar a documentação para utilizar o mesmo e adicionar novos comando para adicionar usuário, etc... |
@@ -1,18 +1,23 @@ | |||
<?php | |||
|
|||
include __DIR__.'/../bootstrap.php'; |
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.
Dúvida: este arquivo não é gerado automaticamente pelo Composer? Se rodarmos um composer update esse arquivo não vai ser sobregravado?
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.
na verdade esses arquivos nem estão funcionando aqui comigo, eles não deveriam ser versionado e estão, o composer.json não está indicando a pasta ./bin também para mante-los vivos.
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.
Esse arquivo não gerado pelo composer ele é Symfony Console, inclusive eu vou alterar o nome dele para console
para não gerar essa confusão
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.
Certo assim faz mais sentido.
Pra falar a verdade eu acho que o que se aplica melhor neste caso é o esquema de Data Fixtures do Doctrine |
O legal da migrations é que qualquer alteração do banco ele será versionado. Assim futuras alterações é possível fazer a migração facilmente pela ferramente ao invés de ficar comparando na mão. Eu pensei em adicionar comando personalizado no |
Vou fazer algumas alterações e criar uma explicação no |
Fiquei em dúvida se esse PR ainda é necessário? Ou o #81 já resolveu tudo? |
Na vdd o PR #81 só cria um comando para criar a base e adicionar usuário, esse PR focando em integrar migrations. Versionando o banco e facilitando novas alterações no banco. Hoje a noite vou fazer o merge e adicionar no |
Galera, por mim aceitamos esse PR. Alguém tem alguma sugestão ou algo contra? |
@eminetto eu não cheguei a testar as features da PR, mas se funciona e não trás complexidade, manda o merge. Mas acho que não tem nada mais simples do que o |
@jackmakiyama Isso é mais simples. O problema que nas alterações futuras terá q documentar tudo e um passo-a-passo. Com migrations basta executar o comando que ele irá fazer tds as alterações conforme a versão atual do seu banco. |
Alterações realizadas:
cli-config.php
ephpunit_bootstrap.php
não mais utilizadosphpunit.xml
php
ebash