Skip to content

[Aula 01] RA: 6325053 - Matheus Gabriel Correa Braga Viana - #149

Merged
AleTavares merged 1 commit into
AleTavares:mainfrom
Matiasdocs:entregas/aula-01/6325053
Sep 19, 2026
Merged

AleTavares merged 1 commit into
AleTavares:mainfrom
Matiasdocs:entregas/aula-01/6325053

Conversation

@Matiasdocs

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

Avaliação do Trabalho de Fixação — Aula 01

📊 Nota Final: 1,35 / 1,5 (90%)


📋 Análise por Critério

# Critério Peso Status Pontos
1 Repositório público no GitHub com nome correto 10% 0,15
2 Estrutura de pastas organizada 10% 0,15
3 Workflow Git com branches (feature → merge) 15% 0,225
4 Mínimo de 5 commits com mensagens descritivas 15% 0,225
5 Aplicação Express funcional com endpoints 10% 0,15
6 Dockerfile funcional e otimizado 15% ⚠️ 0,135
7 .gitignore e .dockerignore configurados 10% 0,15
8 Evidência de container rodando 10% 0,15
9 README pessoal com aprendizados reais 5% 0,075
AWS Academy Pendente conferência do professor 20%

Total avaliável via PR: 1,35 / 1,5


✅ Pontos Fortes

  1. Repositório bem estruturado: Organização clara com README.md na raiz, .gitignore funcional e pasta aula-01/ bem distribuída.

  2. Workflow Git correto: Branch feature/aula-01-app criada, desenvolvida e mergeada para main — demonstra entendimento do fluxo solicitado.

  3. Commits com padrão Conventional Commits: Mensagens descritivas (feat:, docs:, chore:) — 5+ commits confirmados.

  4. Aplicação Express funcional: server.js implementado com endpoints / e /health conforme especificação, retornando JSON com dados corretos (nome, RA, aula, timestamp).

  5. 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.

  6. 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.

  7. .dockerignore e .gitignore presentes: Configurações apropriadas evitando node_modules em ambos os contextos.


⚠️ Ressalvas

1. Dockerfile — Otimização Parcial (−0,015 pontos)

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 Fortes
1. 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
  • Evidência de execução: ✅ Presente e clara
  • Padrões adotados: ✅ Conventional Commits, estrutura solicitada
  • Autenticidade: ✅ README com reflexão genuína, erro real documentado

AWS Academy (20%): Não verificável via PR — pendente conferência manual do professor conforme protocolo.


Avaliacao automatica gerada por GitHub Actions. A nota final e revisada pelo professor, incluindo o componente AWS Academy.

@AleTavares
AleTavares merged commit 4c65237 into AleTavares:main Sep 19, 2026
1 check passed
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