Consider catching thrown errors from validators
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Activo
- Stack tecnológico
- node.js, typescript
- Área
- cli
Línea de trabajo
El issue no nombra archivos, pruebas ni un punto de entrada. Primero aclara si los errores del validador lanzados deben capturarse y cómo debe informarlos la CLI; después, sopesa esto frente a ocultar errores del validador. Para darlo por terminado, se requiere un comportamiento acordado y pruebas que cubran tanto un fallo de validación controlado como excepciones no intencionadas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We currently do not catch thrown errors from validators since it is expected that you return an Error instead.
This is by design and is fine. It may be worth handling uncaught exceptions too, though, just so the CLI gracefully fails validation rather than exiting.
this is the validation implementor's fault at the end of the day, if it happens, but covering it up could give better DX.
thoughts?
the negative is that it could cover up unintentional errors. today it'll be very obvious the validator has a bug and that wouldn't be the case anymore.
- Lenguaje dominante
- TypeScript
- Estrellas
- 8.1k
- Forks
- 218
- Merge medio
- 1 d 23 h
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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 bombshell-dev/clack
-
Dificultad 3/5 1-2 días Aptitud para principiantes 70/100
bombshell-dev/clack#613 · 4 comentarios ·
-
rfc: accessible modeAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
bombshell-dev/clack#585 · 1 comentario · 1 reacción ·
-
[Bug] Spinner does not run `onCancel` function on `Ctrl + C`Posiblemente ocupada @cyphercodes la tomó hace 101 días. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
bombshell-dev/clack#573 · 1 comentario · 2 reacciones ·
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
bombshell-dev/clack#565 · 1 comentario · 1 reacción ·
-
[Request] Improve path selectionAbiertoenhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
bombshell-dev/clack#564 · 1 reacción ·
Todos los issues de bombshell-dev/clack
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
supadata-ai/mcp#27 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
capricorn86/happy-dom#2485 ·
Los mantenedores suelen responder en 2 días
-
优化导入 OCR 模型选择文件的按钮样式Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
siyuan-note/siyuan#20430 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Albert-Weasker/niubigeo#194 ·
Los mantenedores suelen responder en 1 día