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

Remove usage of pkg_resources #356

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

WilliamJamieson
Copy link
Contributor

pkg_resources was deprecated way back in python 3.10 for removal in python 3.12. Since this package is using dynamic versioning from setuptools_scm it should be loading its version from the setuptools_scm generated file not via pkg_resources.

This PR removes the use of pkg_resources and instead loads version information from the file generated by setuptools_scm as part of package install.

Copy link
Collaborator

@mfixstsci mfixstsci left a comment

Choose a reason for hiding this comment

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

This is great, we have been seeing some issues with packages that have dependencies on pysiaf so I am hoping this alleviates the issues we are seeing. Also, perfect timing because @Witchblade101 and I are planning on performing a release this afternoon so we will get this in too.

@WilliamJamieson
Copy link
Contributor Author

This is great, we have been seeing some issues with packages that have dependencies on pysiaf so I am hoping this alleviates the issues we are seeing. Also, perfect timing because @Witchblade101 and I are planning on performing a release this afternoon so we will get this in too.

Thanks I'm glad to help, I ran into an issue related to this, this morning.

@mfixstsci mfixstsci merged commit ccba629 into spacetelescope:main Nov 25, 2024
5 checks passed
@WilliamJamieson WilliamJamieson deleted the bugfix/pkg_resources branch November 26, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants