panzarino/mlbgame

Stop converting HTTPError to ValueError in mlbgame/data.py

Open

#92 opened on 2018年4月27日

GitHub で見る
 (1 comment) (1 reaction) (0 assignees)Python (509 stars) (116 forks)batch import
enhancementhelp wantedquestion

説明

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?

コントリビューターガイド

技術スタック
python
領域
backendapi
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
understanding of Python exception handlingbasic knowledge of HTTP status codes
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65
調査方針
Look at `mlbgame/data.py` where `HTTPError` is caught and converted to `ValueError`. The issue suggests either letting the original `HTTPError` propagate or defining custom exceptions like `Fatal` or `Retryable`. Check the existing exception hierarchy in the codebase and decide which approach aligns with the library's design. Also review any comments in the issue for further discussion.