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

Suporte a Docker para desenvolvimento local #43

Merged
merged 5 commits into from
Sep 14, 2022

Conversation

C0ffeeL0ver
Copy link
Contributor

Issue #26

@gitpod-io
Copy link

gitpod-io bot commented Sep 14, 2022

@IvoPereira
Copy link
Member

Antes de começar a dar review, este Dockerfile explicitamente é para dev ou prod?

Pergunto isto, porque se calhar para dev queremos talvez um nodemon a correr, e se for para prod queremos talvez especificar a versão da imagem de Node a importar, para evitar termos surpresas com breaking changes.

E se for para dev provavelmente queremos ter um volume para que nos permita sincronizar as alterações locais (talvez usando um docker-compose ajudaria?).

@C0ffeeL0ver
Copy link
Contributor Author

Antes de começar a dar review, este Dockerfile explicitamente é para dev ou prod?

Pergunto isto, porque se calhar para dev queremos talvez um nodemon a correr, e se for para prod queremos talvez especificar a versão da imagem de Node a importar, para evitar termos surpresas com breaking changes.

E se for para dev provavelmente queremos ter um volume para que nos permita sincronizar as alterações locais (talvez usando um docker-compose ajudaria?).

fiz a adptação para dev

Copy link
Member

@IvoPereira IvoPereira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dois pontos extra:

  1. Diria que devíamos definir a variáve de ambiente NODE_ENV=development no Dockerfile.
  2. Num improvement futuro poderíamos considerar multi-stage builds para um Dockerfile para produção.

Dockerfile.dev Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
docker-compose.yaml Outdated Show resolved Hide resolved
Dockerfile.dev Outdated Show resolved Hide resolved
alteração da tag do node para 18.9.0-alpine3.16
criação da variável de ambiente NODE_ENV=development
Copy link
Member

@IvoPereira IvoPereira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bom trabalho! Falta-nos depois criar issues para implementar Dockerfiles para CI e Produção.

@IvoPereira IvoPereira merged commit 3fe8633 into devpt-org:main Sep 14, 2022
@IvoPereira IvoPereira mentioned this pull request Sep 14, 2022
3 tasks
@IvoPereira IvoPereira changed the title Suporte a Docker Suporte a Docker para desenvolvimento local Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants