Now check pep8 in travis + made .travis.yml more readable

aitjcize / PyTox

Python binding for Project-Tox the skype replacement.

GNU General Public License v3.0

96 stars 29 forks source link

Now check pep8 in travis + made .travis.yml more readable #29

Closed aviau closed 9 years ago

aviau commented 9 years ago

@aitjcize It passed :icecream: !

aviau commented 9 years ago

The travis failure has to do with upstream tox changes. You can safely merge this since the use of the environment variable worked as expected.

aviau commented 9 years ago

The last commit prevents from running pep8 in every travis job.

There will now be 5 jobs in travis:

python 2.6
python 2.7
python 3.2
python 3.3
pep8

Also, travis will now use tox to run the tests.

It works as expected but fails because of the tox upstream changes(#31). You can safely merge this.

aitjcize commented 9 years ago

After some thought I decide that we should not include the pep8 test in travis. We don't want to fail the tests just because some indentation does not match the standard of pep8. I think running flake8 manually is good enough for me. But still, thanks for your work :)

aviau commented 9 years ago

Alright :)

I will try to find the time to implement new features ASAP.