-
Notifications
You must be signed in to change notification settings - Fork 64
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
Remove test requirements from install_requires #53
Remove test requirements from install_requires #53
Conversation
* Stop using requirements.txt (use setup.py instead). * Remove nose dependecy (plain unittest instead). * Remove lib.release import from setup.py (simple setup.py instead). * Have rake install and use tox to test all python versions (except 3.2 due to broken pip). * Build and upload universal wheels alongside source dists. * Add missing patterns from .gitignore file. * Update RELEASE docs.
Changes Unknown when pulling adcaeca on tysonclugg:issue/51-test-requirements into ** on campaignmonitor:master**. |
1 similar comment
Changes Unknown when pulling adcaeca on tysonclugg:issue/51-test-requirements into ** on campaignmonitor:master**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome. Thanks Tyson!
tox.ini
Outdated
|
||
[tox] | ||
# pip is broken on py32 | ||
envlist = py23, py24, py25, py26, py27, py30, py31, py33, py34, py35, py36, py37 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove py23, py24 and py25 as they're not supported and it looks like Tox is not testing them correctly. I would expect them to fail (missing a json module).
@katharosada I've removed the old Python versions as requested. 🙂 |
Changes Unknown when pulling ee64741 on tysonclugg:issue/51-test-requirements into ** on campaignmonitor:master**. |
Issue #51: