> 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.

Regards,
Daniel

>
> Signed-off-by: Stephen Finucane <step...@that.guru>
> Cc: Daniel Axtens <d...@axtens.net>
> ---
>  tox.ini | 13 +++----------
>  1 file changed, 3 insertions(+), 10 deletions(-)
>
> diff --git a/tox.ini b/tox.ini
> index 0836cd5..c700b54 100644
> --- a/tox.ini
> +++ b/tox.ini
> @@ -32,16 +32,6 @@ commands = flake8 {posargs} patchwork 
> patchwork/bin/pwclient
>  ignore = E129, F405
>  exclude = ./patchwork/migrations
>  
> -[testenv:lint]
> -basepython = python2.7
> -deps =
> -    pylint
> -    -r{toxinidir}/requirements-prod.txt
> -commands = pylint patchwork --rcfile=pylint.rc
> -
> -[testenv:venv]
> -commands = {posargs}
> -
>  [testenv:coverage]
>  basepython = python2.7
>  deps =
> @@ -54,3 +44,6 @@ commands =
>      coverage run 
> --omit=*tox*,patchwork/tests/*.py,manage.py,patchwork/migrations/*.py \
>           --branch {toxinidir}/manage.py test --noinput patchwork
>      coverage report -m
> +
> +[testenv:venv]
> +commands = {posargs}
> -- 
> 2.7.4

Attachment: signature.asc
Description: PGP signature

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

Reply via email to