matrix-org / matrix-org/python-canonicaljson

There are no tests to ensure that floats fail

Ouverte
#22 8 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug help wanted
Langage dominant
Python
Étoiles
30
Forks
15
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser la sérialisation des nombres et la suite de tests existante dans le dépôt, puis retracez la manière dont les valeurs numériques sont traitées. Ajoutez une couverture de régression pour les nombres à virgule flottante et la plage d’entiers spécifiée, puis exécutez la suite complète de tests afin de confirmer que les nombres invalides échouent sans casser le JSON canonique pris en charge.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
backend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
38/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.