Check doc style and refactor linters configuration

.pylintrc

@@ -18,12 +18,6 @@ persistent=yes
 # usually to register additional checkers.
 load-plugins=
 
-# DEPRECATED
-#include-ids=no
-
-# DEPRECATED
-#symbols=no
-
 
 [MESSAGES CONTROL]
 
@@ -54,7 +48,6 @@ load-plugins=
 disable=R,I0011,I0012,W0613,W0511,W0232,W0142,C1001,W0102
 
 
-
 [REPORTS]
 
 # Set the output format. Available formats are text, parseable, colorized, msvs
@@ -68,7 +61,7 @@ output-format=text
 files-output=no
 
 # Tells whether to display a full report or only the messages
-reports=yes
+reports=no
 
 # Python expression which should return a note less than 10 (10 is the highest
 # note). You have access to the variables errors warning, statement which
@@ -221,6 +214,7 @@ generated-members=REQUEST,acl_users,aq_parent,objects,id,_instance,_class_name,F
 [FORMAT]
 
 # Maximum number of characters on a single line.
+# Django coding style guidelines allow up to 119 characters
 max-line-length=119
 
 # Regexp for a line that is allowed to be longer than the limit.

.travis.yml

@@ -19,8 +19,7 @@ env:
     - RESOLWE_POSTGRESQL_USER=postgres
     - RESOLWE_POSTGRESQL_PORT=5432
 
-# NOTE: Specifying Python versions is just to make the Travis CI's web UI a
-# little prettier
+# NOTE: Explicit Python versions make Travis job description more informative
 matrix:
   include:
     - env: TOX_ENV=py27

docs/CHANGELOG.rst

@@ -15,8 +15,12 @@ Added
 - Support loading JSON from a file if the string passed to the ``basic:json:``
   field is a file.
 
+Changed
+-------
+- Refactor linters, check PEP 8 and PEP 257
+
 Fixed
-=====
+-----
 - Fix pylint warnings
 
 

setup.cfg

@@ -1,3 +1,16 @@
 [bdist_wheel]
 # code is written to work on both Python 2 and Python 3
-universal=1
+universal = 1
+
+[pep8]
+# Django coding style guidelines allow up to 119 characters
+max-line-length = 119
+# Ignore E127: checked by pylint
+# E127 continuation line over-indented for visual indent
+ignore = E127
+exclude = migrations,setup.py
+
+[pydocstyle]
+match-dir = (?!tests|migrations|\.).*
+match = (?!setup).*.py
+add-select = D404

setup.py

@@ -66,28 +66,25 @@
         'pyyaml>=3.11',
         'jsonschema>=2.4.0',
         'six>=1.10.0',
+        'Sphinx>=1.4.6',
     ],
     extras_require = {
         'docs':  [
-            # XXX: Intersphinx URLs are broken when using non-default inventory
-            # (like in Django's case) with Sphinx 1.4.3+
-            # Bug report: https://github.com/sphinx-doc/sphinx/issues/2789
-            'Sphinx==1.4.2',
             'sphinx_rtd_theme',
         ],
         'package': [
             'twine',
             'wheel',
         ],
         'test': [
-            'coverage>=3.7.1',
-            'pep8>=1.6.2',
-            'pylint>=1.4.3',
-            'testfixtures>=4.10.0',
             'check-manifest',
+            'coverage>=4.2',
+            'pycodestyle>=2.0.0',
+            'pydocstyle>=1.0.0',
+            'pylint>=1.6.4',
             'readme_renderer',
             'resolwe-runtime-utils>=1.1.0',
-            'Sphinx==1.4.2',
+            'testfixtures>=4.10.0',
         ],
     },
 

tox.ini

@@ -2,8 +2,8 @@
 envlist = py{27,34,35},docs,linters,packaging
 skip_missing_interpreters = True
 
-# NOTE: Don't use 'deps = .[<extra-requirements>]' tox option since we want Tox
-# to install the package from sdist first
+# NOTE: Don't use 'deps = .[<extra-requirements>]' tox option since we
+# want Tox to install the package from sdist first
 
 [testenv]
 commands =
@@ -30,10 +30,12 @@ ignore_errors = True
 commands =
 # install testing requirements
     pip install .[test]
-# run pylint and only check errors
-    pylint --rcfile .pylintrc --reports=no resolwe
-# run pep8
-    pep8 --config=.pep8rc resolwe
+# run pylint
+    pylint resolwe
+# check PEP 8
+    pycodestyle resolwe
+# check PEP 257
+    pydocstyle resolwe
 
 [testenv:packaging]
 commands =