Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EHSTPCR-1181: fix for get_datalabs_path #3010

Merged
merged 3 commits into from
May 21, 2024

Conversation

jespinosaar
Copy link
Contributor

Dear Astroquery team,
This PR aims to fix a small bug associated to the previous pull request, with our new method, get_datalabs_path. In the end, in our Archive FITS are always retrieved as FITS.GZ and, with this fix, we are ensuring this is always considered. Thanks in advance for yout support!
cc @esdc-esac-esa-int

@jespinosaar
Copy link
Contributor Author

Great, it seems all the checks have passed. Please let me know if you need anything else. Thanks a lot!

@jespinosaar
Copy link
Contributor Author

One question, how are now managing the pre-released versions? are they automatically generated after the PR is closed? thanks for the info!

@bsipocz
Copy link
Member

bsipocz commented May 20, 2024

how are now managing the pre-released versions? are they automatically generated after the PR is closed?

Not at the moment, it's all manual. The machinery is broken, but I hope to get funding for infrastructure overhaul and overall maintenance and make the release process more robust and sustainable.

Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good, thanks!

@bsipocz bsipocz merged commit 7da77a3 into astropy:main May 21, 2024
9 checks passed
@bsipocz
Copy link
Member

bsipocz commented May 23, 2024

One question, how are now managing the pre-released versions? are they automatically generated after the PR is closed? thanks for the info!

I did one upload today that includes this PR.

@jespinosaar
Copy link
Contributor Author

Many thanks @bsipocz !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants