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.

Sign up for GitHub

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

Jump to bottom

Do not use the deprecated pkg-resources package #112

Merged
merged 5 commits into from
Nov 15, 2024
Merged

Do not use the deprecated pkg-resources package #112

merged 5 commits into from
Nov 15, 2024

Conversation

avalentino
Copy link
Contributor

Usage of pkg-resources is replaced with importlib.

@avalentino
Do not use the deprecated pkg-resources package
58dc47d 
Usage of pkg-resources is replaced with importlib.
@avalentino
Linting
1edd8af
jonhealy1
jonhealy1 requested changes Nov 15, 2024
View reviewed changes
Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks for doing this. One change I think. Also, could you add an entry to the CHANGELOG just under the Unreleased section. Thank you.

docs/api.rst Outdated Show resolved Hide resolved
jonhealy1
jonhealy1 approved these changes Nov 15, 2024
View reviewed changes
Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

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

Nice work :)

@jonhealy1 jonhealy1 merged commit f30d0fd into stac-utils:main Nov 15, 2024
6 checks passed
@avalentino avalentino deleted the feature/no-pkg-resources branch November 15, 2024 15:27
@avalentino
Copy link
Contributor Author

Thanks @jonhealy1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonhealy1 jonhealy1 jonhealy1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@avalentino @jonhealy1