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

Include tests in release source tarball #59

Merged
merged 1 commit into from
Apr 23, 2016
Merged

Conversation

EdwardBetts
Copy link
Contributor

A test suite is a very useful thing, it would be great to include it in the pypi release tarball. That way people who download and build the release can run the tests to ensure the library is working.

Linux distributions, such as Debian, base their packages of Python modules on the pypi release. This means the test suite can be run when building a Debian package, and so catch mistakes in the packaging or errors introduced into dependencies.

I've written some more on this topic on the Debian Python mailing list: https://lists.debian.org/debian-python/2016/04/msg00074.html

A test suite is a very useful thing, it would be great to include it in the
pypi release tarball. That way people who download and build the release can
run the tests to ensure the library is working.

Linux distributions, such as Debian, base their packages of Python modules on
the pypi release. This means the test suite can be run when building a Debian
package, and so catch mistakes in the packaging or errors introduced into
dependencies.

I've written some more on this topic on the Debian Python mailing list.

https://lists.debian.org/debian-python/2016/04/msg00074.html
@coveralls
Copy link

Coverage Status

Coverage remained the same at 71.753% when pulling 4368046 on EdwardBetts:master into 47aaa59 on byroot:master.

@byroot
Copy link
Owner

byroot commented Apr 23, 2016

Oh thanks!

@byroot byroot merged commit b267f46 into byroot:master Apr 23, 2016
byroot added a commit that referenced this pull request Apr 23, 2016
Include tests in release source tarball
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.

None yet

3 participants