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

Replace pkg_resources with importlib.metadata #15441

Merged

Conversation

webknjaz
Copy link
Member

@webknjaz webknjaz commented Aug 14, 2024

It's been deprecated for quite a while. Additionally, it's coming from an undeclared runtime dependency of setuptools.

Copy link

sonarcloud bot commented Aug 14, 2024

@webknjaz
Copy link
Member Author

@chrismeyersfsu @TheRealHaoLiu looks like I can't request reviews in this repo, but perhaps you could merge it already?

import pkg_resources

__version__ = pkg_resources.get_distribution('awx').version
except pkg_resources.DistributionNotFound:
Copy link
Member Author

Choose a reason for hiding this comment

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

LOL, this line errors out with a NameError because it never exists prior to the import.

@webknjaz webknjaz force-pushed the maintenance/modern-entry-points branch from 82af425 to b2e6cd8 Compare September 13, 2024 14:46
@TheRealHaoLiu TheRealHaoLiu enabled auto-merge (squash) September 13, 2024 14:47
It's been deprecated for quite a while. Additionally, it's coming
from an undeclared runtime dependency of `setuptools`.
auto-merge was automatically disabled September 13, 2024 17:24

Head branch was pushed to by a user without write access

@webknjaz webknjaz force-pushed the maintenance/modern-entry-points branch from b2e6cd8 to 2d16514 Compare September 13, 2024 17:24
Copy link

sonarcloud bot commented Sep 13, 2024

@TheRealHaoLiu TheRealHaoLiu enabled auto-merge (squash) September 13, 2024 17:29
@TheRealHaoLiu TheRealHaoLiu merged commit e68370f into ansible:devel Sep 13, 2024
19 checks passed
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.

2 participants