Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adicionar Dockerfile de Produção(+Desenvolvimento) #58

Closed
wants to merge 4 commits into from

Conversation

IvoPereira
Copy link
Member

@IvoPereira IvoPereira commented Nov 12, 2022

Este PR adiciona uma proposta de Dockerfile que poderia ser usada em produção.

Adicionei o nodemon nos packages de desenvolvimento e acabei por implementar algumas variações no Dockerfile para que algumas etapas só sejam executadas em produção ou desenvolvimento (p.ex. não instalação de packages de dev em produção).

Numa última etapa verifico se estamos na etapa de produção e instalo o TypeScript para criar o ficheiro de distribuição final.

Este Dockerfile carece certamente ainda de várias melhorias, uma deles seria talvez uma etapa final que apenas continha este ficheiro criado na etapa anterior. No entanto isto invoca outras questões tais como:

  • O que fazer num cenário de ambiente de desenvolvimento? Ao copiarmos meramente os ficheiros de distribuição perdemos os packages de desenvolvimento (p.ex. nodemon).

PS: Sendo este PR aprovado, a ideia seria o Dockerfile.dev ser eventualmente substituído (tal como a sua utilização no docker-compose.yaml), mas estou a considerar senão devemos até deixar ambos separados, até para conseguirmos otimizar processos como os que referi acima sem condicionais.

Todo o feedback é bem-vindo.

Issue relacionado: #26

@IvoPereira IvoPereira self-assigned this Nov 12, 2022
@gitpod-io
Copy link

gitpod-io bot commented Nov 12, 2022

@IvoPereira IvoPereira added the 🛑 fora dos planos Não vai ter continuação label Dec 26, 2022
@IvoPereira
Copy link
Member Author

Confesso não ter ficado satisfeito com esta implementação minha e como tal vou fechar o POC.

Fica aberto a sugestões para outras implementações.

@IvoPereira IvoPereira closed this Dec 26, 2022
@IvoPereira IvoPereira deleted the ivopereira/adicionar-suporte-a-26 branch December 26, 2022 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛑 fora dos planos Não vai ter continuação
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant