Permalink
Browse files

Enable pylint during testing. Closes #42

  • Loading branch information...
1 parent f14e75c commit b22eda7cd67a062f49a0f60b8ac51383e4f8c3a5 @atodorov atodorov committed Dec 14, 2016
Showing with 15 additions and 1 deletion.
  1. +7 −1 .travis.yml
  2. +3 −0 Makefile
  3. +4 −0 README.rst
  4. +1 −0 requirements.txt
View
@@ -2,6 +2,8 @@ after_success:
- coveralls
before_install:
- pip install coveralls
+- if [ -z "$_COMMAND" ]; then export _COMMAND=coverage; fi
+- if [ -z "$_DJANGO" ]; then export _DJANGO=1.10.4; fi
env:
- !!python/unicode '_DJANGO=1.10'
- !!python/unicode '_DJANGO=1.10.2'
@@ -41,6 +43,10 @@ matrix:
python: 3.3
- env: _DJANGO=1.10.4
python: 3.3
+ include:
+ - env: _COMMAND=pylint
+ python: 3.5
+
notifications:
email:
on_failure: change
@@ -50,4 +56,4 @@ python:
- 3.3
- 3.4
- 3.5
-script: make coverage
+script: make $_COMMAND
View
@@ -26,6 +26,9 @@ clean-pyc:
lint:
flake8 setup.py chartit demoproject
+pylint:
+ pylint -rn setup.py chartit demoproject/*.py demoproject/demoproject/*.py
+
test: lint
python runtests.py demoproject
View
@@ -28,6 +28,10 @@ pivoted by specific column(s).
Changelog
=========
+* master
+ * Enable pylint during testing but don't block Travis-CI on failures. Closes
+ `#42 <https://github.com/chartit/django-chartit/issues/42>`_.
+
* 0.2.8 (December 4, 2016)
* ``PivotChart`` and ``PivotDataPool`` **will be deprecated soon**. Both
are marked with deprecation warnings. There is a lot of duplication and
View
@@ -1,5 +1,6 @@
# used for testing
flake8
+pylint
coverage
# test requirements from demoproject/
pytz

0 comments on commit b22eda7

Please sign in to comment.