Skip to content
This repository has been archived by the owner on Nov 15, 2022. It is now read-only.

Implementando migrations #76

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

vitornp
Copy link
Contributor

@vitornp vitornp commented Aug 8, 2014

Alterações realizadas:

  • Removendo cli-config.php e phpunit_bootstrap.php não mais utilizados
  • Correção nas configuração do phpunit.xml
  • Integração e documentação do Doctrine migrations
  • Adicionando opção de executar comandos através do php e bash

@vitornp
Copy link
Contributor Author

vitornp commented Aug 8, 2014

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';
Copy link
Contributor

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?

Copy link
Contributor

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.

Copy link
Contributor Author

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

Copy link
Contributor

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.

@eminetto
Copy link
Contributor

eminetto commented Aug 8, 2014

Pra falar a verdade eu acho que o que se aplica melhor neste caso é o esquema de Data Fixtures do Doctrine
Porque o que precisamos é uma forma de carregar dados na base de dados, e usar o schema-tool para criar a estrutura.

@vitornp vitornp mentioned this pull request Aug 8, 2014
@vitornp
Copy link
Contributor Author

vitornp commented Aug 8, 2014

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 ./bin/doctrine (que será ./bin/console) para a carga inicial

@vitornp
Copy link
Contributor Author

vitornp commented Aug 8, 2014

Vou fazer algumas alterações e criar uma explicação no README.md de como irá funcionar o esquema, assim fica mais claro as alterações.

@vitornp vitornp changed the title Implementando migrations Implementando migrations (#66) Aug 8, 2014
@vitornp vitornp changed the title Implementando migrations (#66) Implementando migrations Aug 8, 2014
@eminetto
Copy link
Contributor

Fiquei em dúvida se esse PR ainda é necessário? Ou o #81 já resolveu tudo?

@vitornp
Copy link
Contributor Author

vitornp commented Aug 11, 2014

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 README.md uma explicação de como usar a ferramenta e talz.

@vitornp
Copy link
Contributor Author

vitornp commented Aug 12, 2014

@eminetto Fiz o merge e adicionado um documentação sobre migrations.

Bem provável que as alterações conflite com o PR #82

@eminetto
Copy link
Contributor

Galera, por mim aceitamos esse PR. Alguém tem alguma sugestão ou algo contra?

@jackmakiyama
Copy link
Contributor

@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 ./vendor/bin/doctrine orm:schema-tool:create.

@vitornp
Copy link
Contributor Author

vitornp commented Aug 20, 2014

@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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants