Ignite subcommand doesn't properly handle fails
Los mantenedores suelen responder en 3 días
Evaluación
Este issue todavía no se ha evaluado.
Descripción
**Requester: @ETSells **
Describe the solution you'd like
ign_subcmd_execute returns a status based on the success of subcommand execution. We do not check this return, but we should for safety.
Check this return, and fail-fast if it is not IGN_OK.
Not marked as a bug as this path has not been reached in testing.
Describe alternatives you've considered
N/A
Additional context
Discovered in https://github.com/SunDevilRocketry/test/pull/60#discussion_r3121509870.
Points of Contact
FC Rev 2 Project Lead: @NArmistead
- Lenguaje dominante
- C
- Estrellas
- 8
- Forks
- 6
- Merge medio
- 2 d 3 h
- PR fusionados (30 d)
- 7
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de SunDevilRocketry/Flight-Computer-Firmware
-
Sensor: Set magnetometer ODR to 30 HzPosiblemente ocupada @NArmistead la tomó hace 13 días. Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
SunDevilRocketry/Flight-Computer-Firmware#347 · 1 asignado ·
Los mantenedores suelen responder en 3 días
-
Driver: Integrate existing driversPosiblemente ocupada @MasterUser43 la tomó hace 6 días. Abiertoenhancement
SunDevilRocketry/Flight-Computer-Firmware#363 · 1 comentario · 4 asignados ·
Los mantenedores suelen responder en 3 días
-
EPIC: Time-varying sensor fusionAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
SunDevilRocketry/Flight-Computer-Firmware#346 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
SQA: Define high priority files with low-level requirements and higher coverage requirementPosiblemente ocupada @ETSells la tomó hace 15 días. Abiertoadvanced documentation enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
SunDevilRocketry/Flight-Computer-Firmware#342 · 1 asignado ·
Los mantenedores suelen responder en 3 días
-
EPIC: v2.8.0 major refactors and breaking changesPosiblemente ocupada @NArmistead la tomó hace 21 días. Abiertoadvanced Epic refactor
SunDevilRocketry/Flight-Computer-Firmware#341 · 2 asignados ·
Los mantenedores suelen responder en 3 días
Todos los issues de SunDevilRocketry/Flight-Computer-Firmware
Issues similares
-
Add `CPC_TAKE_TILL_CHARS`Abiertodifficulty: easy
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ChefKissInc/Inferno#319 ·
-
llvm-tools
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
llvm/llvm-project#235522 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100