On 2016-10-31 20:57, Daniel Axtens wrote:
pylint doesn't work very well with Django, and this is broken.

Interesting. I've been carrying the following local patch:

diff --git a/tox.ini b/tox.ini
index 0836cd52ae1d..940daf01e29e 100644
--- a/tox.ini
+++ b/tox.ini
@@ -36,7 +36,7 @@ exclude = ./patchwork/migrations
 basepython = python2.7
 deps =
     pylint
-    -r{toxinidir}/requirements-prod.txt
+    -r{toxinidir}/requirements-test.txt
 commands = pylint patchwork --rcfile=pylint.rc

 [testenv:venv]

That gets things going, but I haven't followed-up on the list of things
it throws up, so if you think the list is useless I'm happy to go with
your patch.

Yeah, it gets things working but there are a lot of false positives. If this is something someone's using then I guess we could keep it (after applying your patch, that is)? In any case, I've the other patches in this series applied.

Stephen
_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork

Reply via email to