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

doctrine na pasta ./bin #80

Open
Open

Description

@jackmakiyama

Os arquivos doctrine dessa pasta não deve ser versionado e também o composer não está indicando a pasta como bin-dir também.

A não ser que ele tenha sido montado, que pode ser o caso.

@eminetto pode confirmar isso?

Se ele não foi escrito por vocês eu vou remover e indicar a pasta bin-dir no composer.
Se foi vou só mudar o include bootstrap.php para include cli-config.php, para que ele funcione corretamente.

Activity

  1. eminetto commented on Aug 8, 2014

    @eminetto
    Contributor

    Pior q eu não tenho certeza disso. Tem como você fazer um teste? Tentar remover o arquivo e fazer um composer update? Para ver se ele cria novamente e como cria?

  2. jackmakiyama commented on Aug 8, 2014

    @jackmakiyama
    ContributorAuthor

    Não cria, no composer.json não está sendo indicado o diretório e esse ./bin/doctrine inclui o bootstrap da aplicação de vocês não o autoload do composer indicando que foi feito e não gerado.

    Se ele não tiver nenhuma ação a mais que o ./vendor/bin/doctrine acho que seria melhor só indicar o bin-dir para o composer.

    O que me diz?

  3. vitornp commented on Aug 8, 2014

    @vitornp
    Contributor

    Na verdade esse arquivo não é o gerado pelo composer e sim um Symfony Console.

    Com a tarefa de migrations esse arquivo será alterado para console e ainda está em discussão tem um PR #76 se vc quiser acompanhar e ajduar

  4. jackmakiyama commented on Aug 8, 2014

    @jackmakiyama
    ContributorAuthor

    @vitornp estou por dentro da PR, vamos esperar ela desenrolar então antes de executar esse aqui.
    Não conheço o Symfony Console, mas ainda acho que esse arquivo não deve ser versionado, ainda mais se ele tem a mesma função que o ./vendor/bin/doctrine que é o indicado no README para ser usado. E até penso que ele nem precisa existir já que o indicado é usar o que está dentro da pasta vendor.

  5. vitornp commented on Aug 8, 2014

    @vitornp
    Contributor

    @jackmakiyama ok, vi que fez um comentário lá eu dei uma breve explicação sobre ele. Se você quiser mais informação pode dar uma olhada na documenteção dele http://symfony.com/doc/current/components/console/introduction.html

  6. jackmakiyama commented on Aug 8, 2014

    @jackmakiyama
    ContributorAuthor

    Certo, antes ele tinha a mesma função que o gerado pelo composer o tornando redundante. Agora faz mais sentido se renomear como citou lá. Vamos aguardar o andamento para ver como ele vai funcionar e fechar aqui caso necessário.

  7. vitornp commented on Aug 8, 2014

    @vitornp
    Contributor

    ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions