clearer errors

Abierto Apto para principiantes
#46 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
68/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
zig
Área
developer-experience

Línea de trabajo

Reproduce el ejemplo con zig build build-lib:bindings y luego inspecciona src/js/wrap_function.zig alrededor de la línea 180, donde se informa el error convertArg: unsupported type bool. Mejora el diagnóstico para que identifique con mayor claridad el argumento de la función visible para el usuario y el valor no compatible, y después vuelve a ejecutar la compilación para confirmar el error resultante.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Right now it's a bit devex unfriendly if you use a non js wrapped value as an arg to a function signature. For example:

pub fn foo(self: *const Bar, my_bool: bool)  !void { ... }

The error, when you try to build:

➜ zig build build-lib:bindings
build-lib:bindings
└─ install bindings
   └─ compile lib bindings Debug native 1 errors
zig-pkg/zapi-2.2.0-rIqzUXBaBADUsvvvB2N5kOBdLEO0rGPhowTqmIXe3qVh/src/js/wrap_function.zig:180:5: error: convertArg: unsupported type bool
    @compileError("convertArg: unsupported type " ++ @typeName(T));
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

it does not tell you exactly where/what went wrong, but only something internally that went wrong, and unless you're familiar with zapi the error can be quite confusing.

Lenguaje dominante
Zig
Estrellas
4
Forks
4
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de ChainSafe/zapi

Todos los issues de ChainSafe/zapi

Issues similares

Más issues de DevTools

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.