panzarino/mlbgame

Stop converting HTTPError to ValueError in mlbgame/data.py

Open

#92 geöffnet am 27. Apr. 2018

Auf GitHub ansehen
 (1 Kommentar) (1 Reaktion) (0 zugewiesene Personen)Python (116 Forks)batch import
enhancementhelp wantedquestion

Repository-Metriken

Stars
 (509 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Finde den try except Block in mlbgame/data.py, der HTTPError abfängt und ValueError auslöst. Ändere ihn so, dass entweder der ursprüngliche HTTPError erneut ausgelöst wird oder eine benutzerdefinierte Ausnahme definiert wird. Stelle sicher, dass bestehende Tests bestehen, und erwäge das Hinzufügen neuer Tests für das neue Verhalten.
Tech Stack
python
Domain
api
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75