matrix-org / matrix-org/python-canonicaljson

There are no tests to ensure that floats fail

Abierto
#22 8 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug help wanted
Lenguaje dominante
Python
Estrellas
30
Forks
15
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

As per spec:

>Numbers in the JSON must be integers in the range `[-(2**53)+1, (2**53)-1]`.

There isn't any code that checks for floats, either.

Guía de contribución

Abrir la guía de contribución

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.

Línea de trabajo

Comienza localizando la serialización de números y el conjunto de pruebas existente en el repositorio; después, rastrea cómo se manejan los valores numéricos. Añade cobertura de regresión para los números de coma flotante y el rango de enteros especificado, y ejecuta el conjunto completo de pruebas para confirmar que los números no válidos fallan sin romper el JSON canónico compatible.

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

Evaluación

Stack tecnológico
python
Área
backend
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.