From acc078e7f4dd47ed1a9587217c8562f79b6bd0db Mon Sep 17 00:00:00 2001 From: Jason Madden Date: Fri, 11 Mar 2016 07:48:42 -0600 Subject: [PATCH] Seems to be gevent.core that's causing the crash. --- .landscape.yml | 4 ++++ .pylintrc | 7 ++++--- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/.landscape.yml b/.landscape.yml index dd44df5ab..a88efb4e9 100644 --- a/.landscape.yml +++ b/.landscape.yml @@ -30,6 +30,10 @@ ignore-paths: - gevent/_tblib.py # likewise - greentest/six.py + # This triggers https://github.com/PyCQA/pylint/issues/846 on Travis, + # but the file is really small, so it's better to skip this one + # file than disable that whole check. + - gevent/core.py # sadly, this one is complicated - setup.py - greentest/getaddrinfo_module.py diff --git a/.pylintrc b/.pylintrc index acda5e1b6..22a561e88 100644 --- a/.pylintrc +++ b/.pylintrc @@ -22,10 +22,11 @@ # too-many-arguments, ; these are almost always because that's what the stdlib does # redefined-builtin, ; likewise: these tend to be keyword arguments like len= in the stdlib # undefined-all-variable, ; XXX: This crashes with pylint 1.5.4 on Travis (but not locally on Py2/3 -# ; or landscape.io on Py3). The file causing the problem is unclear. +# ; or landscape.io on Py3). The file causing the problem is unclear. UPDATE: identified and disabled +# that file. +# see https://github.com/PyCQA/pylint/issues/846 # useless-suppression: the only way to avoid repeating it for specific statements everywhere that we # do Py2/Py3 stuff is to put it here. Sadly this means that we might get better but not realize it. -# see https://github.com/PyCQA/pylint/issues/846 disable=wrong-import-position, wrong-import-order, missing-docstring, @@ -42,7 +43,7 @@ disable=wrong-import-position, too-many-arguments, redefined-builtin, useless-suppression, -# undefined-all-variable + undefined-all-variable [FORMAT]