Drop pylint

Followup 29e6131

Author: philpep

.travis.yml

@@ -8,9 +8,6 @@ install:
   - pip install tox
 jobs:
   include:
-    - stage: lint
-      script: tox -e pylint
-      name: pylint
     - script: tox -e flake8
       name: flake8
     - script: tox -e check-manifest

MANIFEST.in

@@ -7,7 +7,6 @@ include doc/Makefile
 include tox.ini
 include Makefile
 include README.rst CONTRIBUTING.rst CHANGELOG.rst
-include pylintrc
 include MANIFEST.in
 include ansible.cfg
 include dev-requirements.txt

testinfra/backend/winrm.py

@@ -22,9 +22,7 @@
 
 import winrm.protocol
 
-# pylint: disable=no-member
 _find_unsafe = re.compile(r'[^\w@%+=:,./-]', re.ASCII)
-# pylint: enable=no-member
 
 
 # (gtmanfred) This is copied from pipes.quote, but changed to use double quotes

testinfra/host.py

@@ -80,7 +80,7 @@ def run_expect(self, expected, command, *args, **kwargs):
         :param expected: A list of expected exit status
         :raises: AssertionError
         """
-        __tracebackhide__ = True  # pylint: disable=unused-variable
+        __tracebackhide__ = True
         out = self.run(command, *args, **kwargs)
         assert out.rc in expected, (
             'Unexpected exit code %s for %s' % (out.rc, out))
@@ -99,7 +99,7 @@ def check_output(self, command, *args, **kwargs):
         :returns: stdout without trailing newline
         :raises: AssertionError
         """
-        __tracebackhide__ = True  # pylint: disable=unused-variable
+        __tracebackhide__ = True
         out = self.run(command, *args, **kwargs)
         assert out.rc == 0, (
             'Unexpected exit code %s for %s' % (out.rc, out))

testinfra/modules/addr.py

@@ -20,7 +20,6 @@ def __init__(self, addr, port):
 
     @property
     def is_reachable(self):
-        # pylint: disable=protected-access
         if not self._addr._host.exists('nc'):
             # Fallback to bash if netcat is not available
             return self._addr.run_expect(

testinfra/modules/ansible.py

@@ -37,7 +37,6 @@ def __init__(self, result):
 def need_ansible(func):
     @functools.wraps(func)
     def wrapper(self, *args, **kwargs):
-        # pylint: disable=protected-access
         if not self._host.backend.HAS_RUN_ANSIBLE:
             raise RuntimeError((
                 "Ansible module is only available with ansible "
@@ -101,7 +100,6 @@ class Ansible(InstanceModule):
     # Installs neovim and vim.
 
     """
-    # pylint: disable=self-assigning-variable
     AnsibleException = AnsibleException
 
     @need_ansible

testinfra/modules/package.py

@@ -61,7 +61,6 @@ def __repr__(self):
         return "<package %s>" % (self.name,)
 
     @classmethod
-    # pylint: disable=too-many-return-statements
     def get_module_class(cls, host):
         if host.system_info.type == 'windows':
             return ChocolateyPackage

testinfra/modules/service.py

@@ -42,7 +42,6 @@ def is_enabled(self):
         """Test if service is enabled"""
         raise NotImplementedError
 
-    # pylint: disable=too-many-return-statements
     @classmethod
     def get_module_class(cls, host):
         if host.system_info.type == "linux":

testinfra/modules/socket.py

@@ -165,7 +165,6 @@ def get_listening_sockets(cls):
         ['tcp://0.0.0.0:22', 'tcp://:::22', 'unix:///run/systemd/private', ...]
         """
         sockets = []
-        # pylint: disable=protected-access
         for sock in cls(None)._iter_sockets(True):
             if sock[0] == "unix":
                 sockets.append("unix://" + sock[1])

testinfra/plugin.py

@@ -9,7 +9,6 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-# pylint: disable=redefined-outer-name
 
 import logging
 import shutil