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

Changes to run all projects correctly #7

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

cjatoba
Copy link

@cjatoba cjatoba commented Nov 25, 2024

Ao tentar executar o projeto verifiquei que não estava funcionando, foi necessário realizar alguns ajustes, esta PR contempla estes ajustes necessários.

@cjatoba cjatoba force-pushed the changes-to-run-all-projects-correctly branch from 202dd12 to bee0e5e Compare November 25, 2024 05:57
@argentinaluiz
Copy link
Contributor

Olá Clayton @cjatoba , tudo bem?

Muito obrigado pelas contribuições! Temos alguns detalhes pra ajustar, fiz os comentários usando o próprio Github.

Quando terminar, só me avisar que faço o merge da pull request

@cjatoba
Copy link
Author

cjatoba commented Nov 27, 2024

Olá @argentinaluiz tudo blzal,

Esses são todos os ajustes necessários.
Aproveitando, parabéns pelo conteúdo da imersão, foi bastante agregador!

@argentinaluiz
Copy link
Contributor

@cjatoba Muito obrigado pelo feedback! Na semana que vem iremos outra semana Full Cycle, começarão as lives e depois na outra semana, a semana do evento.

Você chegou a subir a atualização removendo os container_name dos docker-compose.yaml?

@cjatoba
Copy link
Author

cjatoba commented Nov 28, 2024

@cjatoba Muito obrigado pelo feedback! Na semana que vem iremos outra semana Full Cycle, começarão as lives e depois na outra semana, a semana do evento.

Você chegou a subir a atualização removendo os container_name dos docker-compose.yaml?

Não chegou a aparecer aqui as sugestões, seriam apenas os container_name? Vou ajustar aqui!

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