Skip to content

Remove pkg_resources usage#51

Draft
Mirochill wants to merge 1 commit into
webrecorder:mainfrom
Mirochill:fix-50-remove-pkg-resources
Draft

Remove pkg_resources usage#51
Mirochill wants to merge 1 commit into
webrecorder:mainfrom
Mirochill:fix-50-remove-pkg-resources

Conversation

@Mirochill

Copy link
Copy Markdown

Summary

  • Remove unused pkg_resources imports from the CLI and validation modules.
  • Replace pkg_resources.get_distribution(wacz).version with importlib.metadata.version(wacz).
  • Add regression coverage for package version lookup.

Fixes #50

Validation

  • git diff --check HEAD~1..HEAD
  • rg -n pkg_resources|resource_filename|get_distribution|DistributionNotFound|load_entry_point wacz tests setup.py (no matches)
  • Not run locally: test suite

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.

ModuleNotFoundError: No module named 'pkg_resources'

1 participant