astropy / astropy/astroquery

MAINT: retire/rename exceptions.TimeoutError in favour of requests

Open
#3,037 0 comments 1 reaction 0 assignees View on GitHub
cleanup
Dominant language
Python
Stars
791
Forks
451
Avg merge
1d 3h
Merged PRs (30d)
4

Description

There is a TimeoutError defined in requests, I would think it may make sense to either remote astroquery's TimeoutError or rename it to be `AstroqueryTimeoutError` instead

Contributor guide

Open the contributing guide

Research direction

No file or test is named. Locate the definitions and usages of astroquery's TimeoutError and compare them with requests' exception, then determine whether removal or renaming is intended and verify that the chosen public exception behavior is consistent across affected code.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend-api-design
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.