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

Just adding a small line in setup.py so that you can just run tests by running 'python setup.py test' #1

Closed
wants to merge 2 commits into from

Conversation

lowks
Copy link

@lowks lowks commented Jun 8, 2014

Your code is superb and needs very little help actually, it doesn't even need any pep8 help. A breath of fresh air! May I just suggest a small addition to your setup.py to allow tests be run via python setup.py test

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e9ea42a on lowks:master into bcfa1c2 on ryankanno:master.

@ryankanno
Copy link
Owner

Thanks for this, but this looks to only be running the nose tests. Is there anyway to include the doctests as well (take a peek to at the flags I'm using in the Makefile).

@lowks
Copy link
Author

lowks commented Jun 14, 2014

I have included it in and it runs the tests, but as of how much it can be close to the Makefile, I don't realy know as development into nosetests inttegration seems to have been abandoned or deprecated.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling f81cd92 on lowks:master into bcfa1c2 on ryankanno:master.

@ryankanno
Copy link
Owner

I've enabled python setup.py nosetests instead of test which supports the coverage and with-doctest plugin (9ed8a1b) so I'm going to close this. Thanks for bringing this to my atttention though!

@ryankanno ryankanno closed this Jun 22, 2014
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