Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove dependency on pkg_resources #307

Merged
merged 1 commit into from
May 21, 2021
Merged

Conversation

lkollar
Copy link
Contributor

@lkollar lkollar commented May 21, 2021

The only reason we needed this was to look at the version metadata and
the installation location. We can do this with importlib.metadata
which is part of Python since 3.8 and is significantly faster to import
than pkg_resources.

Closes: #306

@lkollar lkollar requested a review from mayeut May 21, 2021 08:41
@lkollar lkollar self-assigned this May 21, 2021
@lkollar lkollar force-pushed the remove-pkg_resources branch from 4c47f8c to 6d091a7 Compare May 21, 2021 08:45
The only reason we needed this was to look at the version metadata and
the installation location. We can do this with `importlib.metadata`
which is part of Python since 3.8 and is significantly faster to import
than `pkg_resources`.
@lkollar lkollar force-pushed the remove-pkg_resources branch from 6d091a7 to 6bdc458 Compare May 21, 2021 08:49
@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #307 (6bdc458) into master (4b05be8) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
+ Coverage   91.24%   91.28%   +0.03%     
==========================================
  Files          20       20              
  Lines        1097     1102       +5     
  Branches      237      237              
==========================================
+ Hits         1001     1006       +5     
  Misses         54       54              
  Partials       42       42              
Impacted Files Coverage Δ
auditwheel/main.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b05be8...6bdc458. Read the comment docs.

Copy link
Member

@mayeut mayeut left a comment

Choose a reason for hiding this comment

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

LGTM

@mayeut mayeut merged commit 0c59594 into pypa:master May 21, 2021
@mayeut mayeut mentioned this pull request Aug 28, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants