You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Aplicação Express funcional: server.js implementado com endpoints / e /health conforme especificação, retornando JSON com dados corretos (nome, RA, aula, timestamp).
Evidência clara de execução: Output do docker ps e resposta da API capturados e documentados no entrega.md, demonstrando container rodando com sucesso.
README pessoal autêntico: Não é template — descreve genuinamente a compreensão do aluno sobre Git (branches, save points), Docker (imagem vs. container) e relata dificuldade real (arquivo .txt salvo por acidente). Reflexão crítica presente.
.dockerignore e .gitignore presentes: Configurações apropriadas evitando node_modules em ambos os contextos.
FROM node:20-alpine
WORKDIR /app
COPY package*.json ./
RUN npm install --production
COPY . .
EXPOSE 3000
CMD ["node", "server.js"]
Observação: Dockerfile está funcional e usa node:20-alpine (otimização de tamanho). Porém, não há uso de:
USER para rodar container sem privilégios root (segurança)
Health check nativo (HEALTHCHECK)
Otimização com .dockerignore mencionada mas não verificável no diff do Dockerfile em si
Impacto: Mínimo — 0,015 ponto descontado por não explorar otimizações adicionais de boas práticas.
2. Informação no entrega.md
O arquivo está completo e bem preenchido. Nenhuma ressalva significativa.
📝 Parecer Final para o PR
## ✅ APROVADO**Aluno:** Matheus Gabriel Correa Braga Viana
**RA:** 6325053
**Nota:** 1,35 / 1,5 (90%)
---### Resumo da Avaliação
Entrega **completa e funcional**. O aluno demonstrou compreensão sólida dos dois pilares da aula (Git e Docker):
#### ✅ Critérios Atendidos- ✅ Repositório público (`unifaat-devops-portfolio`) com estrutura clara e padrão
- ✅ Workflow Git correto: branch `feature/aula-01-app` → merge para `main`- ✅ 5+ commits com mensagens em Conventional Commits (`feat:`, `docs:`, etc.)
- ✅ Aplicação Express funcional com endpoints `/` e `/health` retornando dados esperados
- ✅ Dockerfile funcional, testado e rodando com sucesso
- ✅ `.gitignore` e `.dockerignore` presentes e configurados corretamente
- ✅ Evidência clara de container rodando (`docker ps` + resposta da API)
- ✅ README pessoal com reflexões autênticas (não template) — boa explanação sobre branches como "save points", diferença entre imagem e container, e dificuldade real documentada
#### ⚠️ Observações-**Dockerfile**: Funcional e otimizado (alpine). Poderia explorar `USER`, `HEALTHCHECK` e detalhes de segurança em futuras aulas para nota máxima.
#### 📌 Pontos Fortes1. Código limpo e bem organizado
2. Autoconhecimento demonstrado no README (relata erros encontrados e como resolveu)
3. Disciplina no uso de padrões de commit
4. Entrega documentada de forma clara
---### Resultado: **APROVADO** ✅**Nota automática (via PR):** 1,35 / 1,5
**Pendente:** AWS Academy (20%) — conferência manual do professor
---*Ótimo começo no portfólio! Continue assim nas próximas aulas. Dica: na próxima, experimente adicionar `HEALTHCHECK` no Dockerfile e rodar com `docker-compose` para já ir se preparando para aula 02.*
🔍 Observações Adicionais
Portfolio encontrado: ✅ Sim, público e acessível
Código verificável: ✅ Sim, todos os arquivos presentes
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.