Skip to content

Refactor/docker container#1090

Merged
jvictordev1 merged 8 commits into
mainfrom
refactor/docker-container
May 20, 2026
Merged

Refactor/docker container#1090
jvictordev1 merged 8 commits into
mainfrom
refactor/docker-container

Conversation

@lucasn4s
Copy link
Copy Markdown
Collaborator

Por favor, verifique se o seu pull request está de acordo com o checklist abaixo:

  • A implementação feita possui testes (Caso haja um motivo para não haver testes/haver apenas testes de snapshot, descrever abaixo)
  • A documentação no mdx foi feita ou atualizada, caso necessário
  • O eslint passou localmente

1 - Resumo

  • AJusta configuração do docker para desenvolvimento dentro do container.

2 - Tipo de pull request

  • 🧱 Novo componente
  • ✨ Nova feature ou melhoria
  • 🐛 Fix
  • 👨‍💻 Refatoração
  • 📝 Documentação
  • 🎨 Estilo
  • 🤖 Build ou CI/CD

3 - Esse PR fecha alguma issue? Favor referenciá-la

#1077

4 - Quais são os passos para avaliar o pull request?

  • Siga os passos do README para subir o container do projeto;
  • Verifique que o projeto continua se comportando corretamente;
  • Tente rodar o projeto sem o container para verificar que tudo continua funcionando para esta opção;
  • Deixe seu like e se inscreva no canal.

5 - Imagem ou exemplo de uso:

6 - Esse pull request adiciona breaking changes?

  • Sim
  • Não

@github-actions github-actions Bot added 🐛 Bug Algo não está funcionando 📃 Documentação Melhorias ou adição de documentação labels May 18, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 18, 2026

Greptile Summary

Este PR refatora o ambiente Docker de desenvolvimento. As principais mudanças são:

  • Troca a imagem Docker pré-existente por build local via .docker/Dockerfile.
  • Atualiza o compose para servir a documentação na porta 5173.
  • Adiciona setup.sh para escolher porta e subir o container.
  • Atualiza o README com o novo fluxo de desenvolvimento em container.

Confidence Score: 2/5

Este PR ainda pode quebrar o fluxo principal de desenvolvimento em container.

  • O setup pode deixar docker-compose.yml alterado quando o compose falha ou é interrompido.
  • O caminho documentado pode falhar em macOS por causa do sed -i.
  • Os comandos via on-server.sh podem falhar em ambientes com apenas Docker Compose v2.
  • O build Docker não usa o ignore file criado para esse contexto.

setup.sh, on-server.sh, .docker/Dockerfile e .docker/.dockerignore.

Important Files Changed

Filename Overview
setup.sh Adiciona o script de setup com detecção de porta e mutação temporária do compose.
docker-compose.yml Passa a construir a imagem localmente e executar o VitePress em 0.0.0.0.
.docker/Dockerfile Instala dependências durante o build e inicia o servidor de docs por padrão.
.docker/.dockerignore Define exclusões para o build Docker, mas está fora da raiz do contexto usado.
README.md Documenta o novo fluxo com setup.sh e on-server.sh.

Reviews (1): Last reviewed commit: "Atualiza readme para utilizar docker" | Re-trigger Greptile

Comment thread setup.sh Outdated
Comment thread setup.sh Outdated
Comment thread setup.sh
Comment thread .docker/Dockerfile Outdated
Comment thread .dockerignore Outdated
@ezequias21
Copy link
Copy Markdown
Contributor

Quando tento executar algo com o ./on-server.sh:

image

Comment thread docker-compose.yml Outdated
Comment thread .docker/Dockerfile Outdated
Comment thread .docker/Dockerfile Outdated
Comment thread .docker/Dockerfile
@ezequias21
Copy link
Copy Markdown
Contributor

Está faltando o working_dir: /usr/src/app no YAML

@jvictordev1 jvictordev1 merged commit 10c9272 into main May 20, 2026
8 checks passed
@jvictordev1 jvictordev1 deleted the refactor/docker-container branch May 20, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Bug Algo não está funcionando 📃 Documentação Melhorias ou adição de documentação

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants