panzarino/mlbgame

Stop converting HTTPError to ValueError in mlbgame/data.py

Open

#92 aperta il 27 apr 2018

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)Python (116 fork)batch import
enhancementhelp wantedquestion

Metriche repository

Star
 (509 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Individua il blocco try except in mlbgame/data.py che cattura HTTPError e solleva ValueError. Modificalo per rilanciare l'HTTPError originale o definire un'eccezione personalizzata. Assicurati che i test esistenti passino e considera l'aggiunta di nuovi test per il nuovo comportamento.
Tech stack
python
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75