panzarino/mlbgame

Stop converting HTTPError to ValueError in mlbgame/data.py

Open

#92 ouverte le 27 avr. 2018

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)Python (116 forks)batch import
enhancementhelp wantedquestion

Métriques du dépôt

Stars
 (509 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Converting HTTPError to ValueError loses valuable information about the failure. If the HTTPError itself was allowed to be raised then callers could look at the status code and determine if they want to try the call again (in the case of a 5xx error) or stop trying (4xx error).

Or maybe it could raise new exceptions defined by the mlbgame library? Fatal or Retryable?

Guide contributeur

Direction de recherche
Localisez le bloc try except dans mlbgame/data.py qui attrape HTTPError et lève ValueError. Modifiez le pour relancer l'HTTPError original ou définir une exception personnalisée. Assurez vous que les tests existants passent et envisagez d'ajouter de nouveaux tests pour le nouveau comportement.
Stack technique
python
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75