Skip to content

Adiciona Dockerfile e job de CI que sobe a imagem - #28

Merged
brolesi merged 1 commit into
mainfrom
feat/docker
Oct 9, 2026
Merged

brolesi merged 1 commit into
mainfrom
feat/docker

Conversation

@brolesi

@brolesi brolesi commented Oct 9, 2026

Copy link
Copy Markdown
Owner

Empilhado sobre #27 (pilha #24 → #25 → #26 → #27 → este). Mergear na ordem, mudando a base para main a cada passo; o CI só roda com base main.

Resumo

  • Dockerfile (python:3.12-slim): sobe a API sem Python local — docker build -t taco . e docker run -p 8000:8000 taco. Copia só api/ e data/processed/; roda como nobody.
  • .dockerignore em allowlist, só para encolher o contexto de build.
  • Job docker no CI: build da imagem, sobe o contêiner e consulta /health e /foods/561. Não havia Docker na máquina onde isto foi escrito — este job é o único teste do Dockerfile, então vale olhar o resultado dele com atenção quando o CI rodar.
  • Versão 1.12.0; README e CHANGELOG atualizados.

Publicação no Kaggle / Hugging Face (resto do item) ficou de fora por decisão: os dados já estão no GitHub, nas Pages, nas releases e no Zenodo.

Testes

  • ruff check ., ruff format --check . ok; pytest 53 passed.
  • Dockerfile não testado localmente (sem Docker); coberto pelo novo job de CI.

🤖 Generated with Claude Code

@brolesi
brolesi force-pushed the feat/valores-diarios branch from 09e5384 to 5f04424 Compare October 9, 2026 22:23
@brolesi
brolesi force-pushed the feat/valores-diarios branch from 5f04424 to deaf9ba Compare October 9, 2026 22:27
Base automatically changed from feat/valores-diarios to main October 9, 2026 22:29
A imagem leva só api/ e data/processed/ e roda sem root. Como não há
Docker na máquina de desenvolvimento, o job `docker` do CI constrói a
imagem, sobe o contêiner e consulta /health e /foods/561.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@brolesi
brolesi merged commit b65a820 into main Oct 9, 2026
3 checks passed
@brolesi
brolesi deleted the feat/docker branch October 9, 2026 22:31
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.

1 participant