-
Notifications
You must be signed in to change notification settings - Fork 45
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
3.0.0: Tests not in sdist tarball on pypi.org #69
Comments
Remove unneeded prepare(). Simplify quoting in file. Add new upstream ticket to track missing tests in sdist tarball: django-cms/django-classy-tags#69 Simplify installation in package() to adhere to Python package guidelines. Remove unneeded tests dir removal (upstream fixed this). git-svn-id: file:///srv/repos/svn-community/svn@1113609 9fca08f4-af9d-4005-b8df-a31f2cc04f65
Remove unneeded prepare(). Simplify quoting in file. Add new upstream ticket to track missing tests in sdist tarball: django-cms/django-classy-tags#69 Simplify installation in package() to adhere to Python package guidelines. Remove unneeded tests dir removal (upstream fixed this). git-svn-id: file:///srv/repos/svn-community/svn@1113609 9fca08f4-af9d-4005-b8df-a31f2cc04f65
Hi @dvzrv! I hope I understand what you're after now. Looking at the current release, the tarball only contains the python of the tests directory. But you need that to include everything so that you can actually run it. So I've updated the manifest to include the requirements and templates so that the tests should be able to function. Now the tarball includes all this; Is that right? |
Thanks! That looks alright now! :) |
@dvzrv Please try 3.0.1 which I've just released. Hopefully that's got what you need now |
Thanks a lot! :) |
Hi! When attempting to package version 3.0.0 I noticed that the sdist tarball on pypi.org still does not contain the tests (see #52 (comment)).
Can they please be added to the sdist tarball? :)
The text was updated successfully, but these errors were encountered: