Code review comment for lp:~adiroiban/pocket-lint/travis-ci

Revision history for this message
Adi Roiban (adiroiban) wrote :

I will use IS_PY3 ... sorry for that. I was not aware of this convention.

I have renamed requires to install_requires so that they will be installed by pip

It looks like the PY3 check is required as this build fails :

With the exception for py3 the test pass :

The code works on Travis-CI

I have no idea why the MERGE markers are listed in this diff. I checked the branch and there are no markers:

Please review latest changes. Thanks!

« Back to merge proposal