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

Move test requirements to requirements_test.txt #190

Closed
dokterbob opened this issue Feb 10, 2016 · 2 comments
Closed

Move test requirements to requirements_test.txt #190

dokterbob opened this issue Feb 10, 2016 · 2 comments

Comments

@dokterbob
Copy link
Collaborator

Currently, webtest and django-webtest are present in the normal requirements because, by default, Django used to run tests for all pluggable apps. However, I do not think this is currently still the case and perhaps these should be moved.

@claudep What do you think about it?

@claudep
Copy link
Collaborator

claudep commented Feb 10, 2016

I'm not familiar with webtest. If we can replace self.app with self.client with no harm, then one less requirement is always better.

@claudep
Copy link
Collaborator

claudep commented Feb 10, 2016

Oh, your question was about moving, not removing. So the answer is "sure", of course.

dsanders11 pushed a commit to dsanders11/django-newsletter that referenced this issue May 2, 2016
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

No branches or pull requests

2 participants