[VizieR] Raise error in case of server-side issues#3632
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3632 +/- ##
==========================================
+ Coverage 73.17% 73.21% +0.04%
==========================================
Files 227 227
Lines 21078 21091 +13
==========================================
+ Hits 15423 15442 +19
+ Misses 5655 5649 -6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Aaarg |
13c4e40 to
421ee61
Compare
|
I'm not sure we should be supporting versions older than that - it was added 3 years ago. @bsipocz what would you say to bumping min-astropy-version? |
Let me see if how quickly we can ship the next release and if it would make sense to keep the versions as is for that release. But in general, I agree that we should bump versions. |
421ee61 to
31a9694
Compare
31a9694 to
1fe5021
Compare
|
No need to bump versions for this PR, I'll loop through the |
Hello astroquery maintainers 👋
This PR intends to solve #3626
It raises an error when a VizieR response contains an
infoelementQUERY_STATUSwith valueERRORand removes the corresponding entry from the cache.