Skip to content

Solucion reto nerve 08 - #56

Merged
Kaia-Alenia merged 1 commit into
nerve-learning:mainfrom
CesarAAR:solucion-reto-nerve-08
Jul 28, 2026
Merged

Solucion reto nerve 08#56
Kaia-Alenia merged 1 commit into
nerve-learning:mainfrom
CesarAAR:solucion-reto-nerve-08

Conversation

@CesarAAR

@CesarAAR CesarAAR commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

📌 ¿Qué reto resuelve este PR?

Reto 08 — Puente HTTP para Nerve (FastAPI)

✅ Checklist

  • Mi código está dentro de la carpeta correcta del reto
  • Agregué un README corto explicando cómo correr mi solución
  • Probé que mi código funciona antes de subirlo

💬 Notas para quien revise

En la tarea menciona que se agregue una ruta 'GET/estado', pero para ser sinceró no encontré una manera logica de resolver esa petición por la naturaleza de nerve. Al no tener funciones con las cuales se pueda saber el estado de la conexión más que solo realizando un "connect"|"get_metrics" o realizando quizas un ping a la ruta tcp ip (que tal vez dicha ip no garantiza que esté conectado a nerve sino que sea de una sesión pasada), no existe tal cual una forma de resolverlo.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Linter Compasivo: Formateo requerido

¡Hola @CesarAAR! Tu código necesita un pequeño ajuste de formato antes de poder hacer merge.

Como tu PR viene desde un fork, no puedo corregirlo automáticamente. Pero es muy fácil de solucionar:

Ejecuta estos comandos en tu repositorio local:

pip install black
black .
git add -A
git commit -m "style: format code with black"
git push

Una vez que hagas push, este check se volverá a ejecutar automáticamente. ¡Ánimo! 💪

@Kaia-Alenia
Kaia-Alenia merged commit c396048 into nerve-learning:main Jul 28, 2026
3 of 4 checks 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