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

Migrate from pkg_resources to importlib.metadata #249

Closed
wants to merge 4 commits into from
Closed

Migrate from pkg_resources to importlib.metadata #249

wants to merge 4 commits into from

Conversation

wwuck
Copy link

@wwuck wwuck commented May 4, 2022

This migrates from deprecated pkg_resources to the stdlib importlib.metadata. Python version is also updated to minimum 3.7 as 3.6 is now EOL.

I'm not sure why flake8 version in poetry.lock was lowered by poetry.

I ran poetry remove setuptools and poetry add --python "<3.8" importlib_metadata to swap the dependency. I am running poetry on python3.10.

Fixes #248

pyproject.toml Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 4, 2022

Codecov Report

Merging #249 (d551827) into master (385c8ed) will not change coverage.
The diff coverage is 100.00%.

❗ Current head d551827 differs from pull request most recent head d6f7de9. Consider uploading reports for the commit d6f7de9 to get more accurate results

@@            Coverage Diff            @@
##            master      #249   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           42        43    +1     
  Branches         8         8           
=========================================
+ Hits            42        43    +1     
Impacted Files Coverage Δ
flake8_eradicate.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@wwuck wwuck requested a review from sobolevn May 4, 2022 14:46
@wwuck
Copy link
Author

wwuck commented Aug 12, 2022

Rebased to fix merge conflicts

@wwuck wwuck closed this Sep 23, 2022
@wwuck wwuck deleted the feature/importlib branch September 23, 2022 01:42
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.

Switch from pkg_resources to importlib.metadata
2 participants