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

avoid the use of pkg_resources when importlib.metadata is available #799

Merged
merged 5 commits into from
Dec 13, 2023

Conversation

d9pouces
Copy link
Contributor

pkg_resources is removed from Python 3.12, and its use raises a warning.
importlib.metadata can be used to get distribution info.

@codecov
Copy link

codecov bot commented Oct 29, 2023

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (0dd41a3) 83.19% compared to head (46f0988) 82.35%.

Files Patch % Lines
pipeline/__init__.py 37.50% 15 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #799      +/-   ##
==========================================
- Coverage   83.19%   82.35%   -0.85%     
==========================================
  Files          33       33              
  Lines        1119     1139      +20     
==========================================
+ Hits          931      938       +7     
- Misses        188      201      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@peymanslh
Copy link
Member

@d9pouces
I've fixed the CI issues in the recent changes
Can you update your PR so I can review and merge it?

@d9pouces
Copy link
Contributor Author

@d9pouces I've fixed the CI issues in the recent changes Can you update your PR so I can review and merge it?

Done!

Can I also expect the merge of #737?

@peymanslh
Copy link
Member

It still has conflict and can't be merged, please rebase your branch with the current master branch

@d9pouces
Copy link
Contributor Author

Conflicts seem to have disappeared and this branch can be merged.

@peymanslh peymanslh force-pushed the remove_pkg_resources branch from d9cd87c to d3a7144 Compare December 13, 2023 23:31
@peymanslh peymanslh merged commit c286019 into jazzband:master Dec 13, 2023
13 of 15 checks passed
@peymanslh
Copy link
Member

Thanks @d9pouces

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