-
Notifications
You must be signed in to change notification settings - Fork 42
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
Polluting site-packages with tests #334
Comments
Hi @karolyi, Thanks for reaching out here with this question! Which library version are you on currently? Best, Jilling |
Hey, latest, of course. |
Hi @karolyi, I wanted to give you a quick update that we've merged the updated setup.py in our latest release! Kind regards, Djoyke |
Thanks. Did you stop releasing on pypi? The latest release on there is12.3.0 from May. I can use github for releases, just let me know if that's the case. |
Yes unfortunately we have not been able to release packages to pypi for a while as we're relying on the (very) understaffed pypi team to recover our account. Let us know if you're able to retrieve the package through Github, if not we will consider providing alternatives. Best, Jilling |
Hey, did you actually test the installation? It still pollutes the site-packages, so not fixed. I already use a whole lot of non-pypi sources (private and other decenralized sources) anyway so using this from github is not an issue, btw. |
To reproduce: pip install git+https://github.com/Adyen/[email protected] then look into your site-packages dir in the venv.
you mean pypi's team is understaffed or you have a dedicated but understaffed team for pypi? |
Hello,
your package pollutes the site-packages directory with tests, upon installing. I was shocked to notice. See below. I've made a test venv for it.
Before installing adyen:
After installing adyen:
The text was updated successfully, but these errors were encountered: