Add travis job

Launch pylint with pylint-django plugins
At this time, only report Warnings and Errors

Remove mysql dependencies because default configuration
use sqlite, we should add dedicated howto to enable it.

Fix error in exportbooks command.
Disable pylint for no-member for save method for FileField
(check pylint-dev/pylint-django#60)

.travis.yml

@@ -0,0 +1,15 @@
+langague: python
+distr: trusty
+sudo: required
+python:
+    - 2.7
+
+before_install:
+    - sudo apt-get install -y python-virtualenv
+    - virtualenv /tmp/venv
+    - source /tmp/venv/bin/activate
+    - pip install -r requirements.pip
+    - pip install -r test-requirements.pip
+
+script:
+    - scripts/run-pylint.sh

books/management/commands/exportbooks.py

@@ -60,7 +60,7 @@ def _handle_csv(self, csvpath):
                 'a_author': book.a_author.a_author.encode('utf-8'),
                 'a_summary': book.a_summary.encode('utf-8')}
             writer.writerow(entry)
-        writer.close()
+        csvfile.close()
 
 
     def handle(self, filepath='', *args, **options):

books/models.py

@@ -140,7 +140,7 @@ def save(self, *args, **kwargs):
                 cover_path = epub_file.get_cover_image_path()
                 if cover_path is not None and os.path.exists(cover_path):
                     cover_file = File(open(cover_path))
-                    self.cover_img.save(os.path.basename(cover_path),
+                    self.cover_img.save(os.path.basename(cover_path), # pylint: disable=no-member
                                         cover_file)
                 epub_file.close()
 

requirements.pip

@@ -1,6 +1,5 @@
 Django==1.4.5
 django-tagging==0.3.1
 django-sendfile==0.3.6
-MySQL-python>=1.2.3
 lxml==3.4.4
 django-taggit==0.14.0

scripts/run-pylint.sh

@@ -0,0 +1,14 @@
+#!/bin/bash
+
+for file in $(git ls-files -- '*.py' | grep -vE 'deploy|fabfile'); do
+    echo "==== $file"
+    pylint --rcfile=pylintrc --load-plugins pylint_django --score=no --reports=no --disable=R,C $file
+    RET=$? # pylint returns bitfield of found issues
+    ERR=$(($RET&3)) # Fatal is 1 and Error is 2
+    if [[ $ERR -ne 0 ]]; then
+        echo "Aborting"
+        exit 1
+    fi
+done
+
+exit 0

test-requirements.pip

@@ -0,0 +1,2 @@
+pylint==1.7.1
+pylint-django==0.7.2