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