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

ENH: Remove unused pkg_resources module import #179

Merged

Conversation

jhlegarreta
Copy link
Contributor

Remove unused pkg_resources module import.

Also, pkg_resources is deprecated in recent setuptools versions.

Fixes:

* Getting build dependencies for wheel...
<string>:5: DeprecationWarning: pkg_resources is deprecated as an API.
See https://setuptools.pypa.io/en/latest/pkg_resources.html
* Installing packages in isolated environment... (numpy>=1.20.0, wheel)
* Building wheel...
<string>:5: DeprecationWarning: pkg_resources is deprecated as an API.
See https://setuptools.pypa.io/en/latest/pkg_resources.html

when trying to upload the package to TestPyPI using $ twine upload --repository testpypi dist/*

Documentation:
https://setuptools.pypa.io/en/latest/pkg_resources.html

Remove unused `pkg_resources` module import.

Also, `pkg_resources` is deprecated in recent `setuptools` versions.

Fixes:
```
* Getting build dependencies for wheel...
<string>:5: DeprecationWarning: pkg_resources is deprecated as an API.
See https://setuptools.pypa.io/en/latest/pkg_resources.html
* Installing packages in isolated environment... (numpy>=1.20.0, wheel)
* Building wheel...
<string>:5: DeprecationWarning: pkg_resources is deprecated as an API.
See https://setuptools.pypa.io/en/latest/pkg_resources.html
```

when trying to upload the package to TestPyPI using
`$ twine upload --repository testpypi dist/*`

Documentation:
https://setuptools.pypa.io/en/latest/pkg_resources.html
@jhlegarreta
Copy link
Contributor Author

Build failures are unrelated.

Merging.

@jhlegarreta jhlegarreta merged commit eb36051 into SlicerDMRI:master Oct 16, 2023
0 of 4 checks passed
@jhlegarreta jhlegarreta deleted the RemoveUnusedPkgResourcesImport branch October 16, 2023 13:21
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.

1 participant