astropy / astroquery

Functions and classes to access online data resources. Maintainers: @keflavich and @bsipocz and @ceb8
http://astroquery.readthedocs.org/en/latest/
BSD 3-Clause "New" or "Revised" License
699 stars 397 forks source link

MAINT: adding Python 3.13 to CI #3088

Closed bsipocz closed 1 month ago

bsipocz commented 1 month ago

If we're compatible out of the box then no changelog entry will be needed...

(locally I see one doctesting whitespace issue, it may actually be an upstream doctestplus issue)

pllim commented 1 month ago

I think the whitespace problem is from Python 3.13 itself. According to https://docs.python.org/3.13/whatsnew/3.13.html, "The compiler now strips common leading whitespace from every line in a docstring."

Is there a reason why you must keep the leading whitespace? If not, would be much simpler to adjust the test for Python 3.13 and later. FYI.

bsipocz commented 1 month ago

A good to know, thanks, I'll change the test then.

bsipocz commented 1 month ago

This now has a pragmatic workaround for https://github.com/python/cpython/issues/81283