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

MAINT: Update _astropy_init to latest version in astropy package-template #608

Merged
merged 2 commits into from
Nov 18, 2023

Conversation

rrjbca
Copy link
Contributor

@rrjbca rrjbca commented Oct 29, 2023

Description

The compatibility workflow is currently failing for the astropy development version with ImportError: cannot import name 'update_default_config' from 'astropy.config.configuration' : https://github.com/skypyproject/skypy/actions/runs/6681172883/job/18155043088#step:6:117

update_default_config was deprected in astropy 5.0 and was recently removed on the main branch in preparation for 6.0: astropy/astropy#15466

This PR updates _astropy_init.py to match the latest version in astropy package-template which no longer requires update_default_config: https://github.com/astropy/package-template/blob/cookiecutter/%7B%7B%20cookiecutter.package_name%20%7D%7D/%7B%7B%20cookiecutter.module_name%20%7D%7D/_%7B%7B%20cookiecutter._parent_project%20%7D%7D_init.py

This patch has been demonstrated to resolve the issue on with the compatibility workflow here: https://github.com/rrjbca/skypy/actions/runs/6682165049/job/18157042681

Once approved I can also backport to v0.5.x and make a release v0.5.3 which will be necessary for the compatibility workflow "latest" job to pass.

Checklist

  • Follow the Contributor Guidelines
  • Write unit tests
  • Write documentation strings
  • Assign someone from your working team to review this pull request
  • Assign someone from the infrastructure team to review this pull request

@rrjbca rrjbca merged commit 8fbe549 into skypyproject:main Nov 18, 2023
8 checks passed
@rrjbca rrjbca deleted the update_astropy_init branch November 18, 2023 12:08
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.

1 participant