Skip to content

Please included tests in pypi release #290

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.

Sign up for GitHub

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

Jump to bottom
Closed
jlec opened this issue Nov 9, 2015 · 2 comments · Fixed by #296
Closed

Please included tests in pypi release #290

jlec opened this issue Nov 9, 2015 · 2 comments · Fixed by #296

Comments

@jlec
Copy link
Contributor

jlec commented Nov 9, 2015

Downstream developers love to test during installation and packaging. Please include tests.
@pelme
Copy link
Member

pelme commented Dec 5, 2015

I would be happy to include tests in the distribution, but have not the motivation to work on this right now. Please send a PR and I will make sure to merge it.

jlec added a commit to jlec/pytest-django that referenced this issue Dec 5, 2015
@jlec
Include tests in pypyi tarball
0fe6b02 
Inlcude test for downstream testing, resolves pytest-dev#290

Signed-off-by: Justin Lecher <[email protected]>
@jlec jlec mentioned this issue Dec 5, 2015
Merged
@pelme pelme closed this as completed in #296 Dec 5, 2015
tomviner pushed a commit to tomviner/pytest-django that referenced this issue Jun 22, 2016
@jlec @tomviner
Include tests in pypyi tarball
742686c 
Inlcude test for downstream testing, resolves pytest-dev#290

Signed-off-by: Justin Lecher <[email protected]>
mfa pushed a commit to aexeagmbh/pytest-django that referenced this issue May 17, 2017
@jlec @mfa
Include tests in pypyi tarball
00b3467 
Inlcude test for downstream testing, resolves pytest-dev#290

Signed-off-by: Justin Lecher <[email protected]>
@blueyed
Copy link
Contributor

blueyed commented Sep 16, 2018 •
edited
Loading

@jlec
This was never fixed really - https://github.com/pytest-dev/pytest-django/pull/296/files included "test", but tests are in tests.
Although tests are included now (https://travis-ci.org/pytest-dev/pytest-django/jobs/429155796#L486) I believe "pytest_django_test" should also be included then.

Would be nice to have this in the source, but not the wheel - does that make sense?

blueyed added a commit to blueyed/pytest-django that referenced this issue Sep 16, 2018
@blueyed
MANIFEST.in: include tests for downstream distros
a479a05 
Ref: pytest-dev#290

[ci skip]
@blueyed blueyed mentioned this issue Sep 16, 2018
Merged
blueyed added a commit to blueyed/pytest-django that referenced this issue Sep 16, 2018
@blueyed
Simplify MANIFEST.in: include tests for downstream distros
f1c4b35 
Ref: pytest-dev#290

[ci skip]
blueyed added a commit to blueyed/pytest-django that referenced this issue Sep 16, 2018
@blueyed
Remove MANIFEST.in: include tests for downstream distros
6ea1111 
Ref: pytest-dev#290

[ci skip]
blueyed added a commit to blueyed/pytest-django that referenced this issue Oct 30, 2018
@blueyed
MANIFEST.in: include tests for downstream distros
70174d8 
Ref: pytest-dev#290
blueyed added a commit that referenced this issue Oct 30, 2018
@blueyed
MANIFEST.in: include tests for downstream distros (#653)
7ebe329 
Ref: #290

Acked-by: Philippe Ombredanne [email protected]
beyondgeeks pushed a commit to beyondgeeks/django-pytest that referenced this issue Sep 6, 2022
MANIFEST.in: include tests for downstream distros (#653)
07c1390 
Ref: pytest-dev/pytest-django#290

Acked-by: Philippe Ombredanne [email protected]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Include tests in pypyi tarball jlec/pytest-django
3 participants
@blueyed @pelme @jlec