search

DeepLcom / deepl-python

Official Python library for the DeepL language translation API.
https://www.deepl.com
MIT License
1.11k stars 80 forks source link

Messy Installation with PIP #66

Closed TurtleWilly closed 1 year ago

TurtleWilly commented 1 year ago

When installing things with PIP, e.g.

pip install deepl --user

things get unnecessarily messy in the package target directory afterwards:

~/Library/Python/3.7/lib/python/site-packages $ find . -depth 1 -newermt '10 min ago'
…
./CHANGELOG.md
./deepl
./deepl-1.13.0.dist-info
./LICENSE
./SECURITY.md
…

The files CHANGELOG.md, LICENSE, and SECURITY.md should be placed inside one of the deepl directories, not into the main package directory. If I was to look for information for deepl, I would check inside the directories, not inside random files where it's not clear instantly where they do belong to. Also if all packages would do it like that only the most recently installed package would have a SECURITY.md, f.ex.

JanEbbing commented 1 year ago

Thanks for reporting this! This will be fixed in an upcoming version.

JanEbbing commented 1 year ago

This has been fixed, installing the current version via pip no longer pollutes the site-packages directory