API: Deprecate vo_conesearch

[pllim]

Close #2027

After merge:

• Open follow-up issue to remove vo_conesearch completely after deprecation period has elapsed. -- no need; we don't normally open such follow-up issues.
• Open follow-up issue to write up a transition guide, if there is a need for it. -- edit by Brigitta: I would also just open this issue if the need arises, not before.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.67%. Comparing base (b1cae75) to head (520556d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3548   +/-   ##
=======================================
  Coverage   72.66%   72.67%           
=======================================
  Files         219      219           
  Lines       20480    20483    +3     
=======================================
+ Hits        14882    14885    +3     
  Misses       5598     5598           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[bsipocz]

This looks good to me.

I'll keep the PR open though for a little while to allow others to chime in if they want.

[pllim]

Just FYI that https://astroconda.org/ needs a SSL cert refresh (cc @jhunkeler) so some vo_conesearch remote data tests are failing because of that.

[bsipocz]

Just FYI that astroconda.org needs a SSL cert refresh (cc @jhunkeler) so some vo_conesearch remote data tests are failing because of that.

Yeap, those are failing for a while. Feel free to skip them altogether; merging this PR kinds of makes off hands off of the module so no need to burn CI resources on it.

[pllim]

The SSL should be fixed now FWIW.

In the future, when this is actually removed, and the removal is released, we want to shut down astroconda.org for good. Hope that is not a problem.

[ManonMarchand]

LGTM. Do we want a link to PyVO's SCS in the warning message?

[pllim]

a link to PyVO's SCS

If the URL isn't terribly long, perhaps. Otherwise, it might make the message too ugly to read. I included link in user doc.