Remove pylint target

This target hasn't been validated in CI. Currently running the target
consistently fails likely because of the quite old version pinned to.

Because hacking/flake8/pep8 is already used, drop the unmaintained
target.

Change-Id: I414b24355a2b750f5ae0ff92613cfbfc1f39f3ff

test-requirements.txt

@@ -9,7 +9,6 @@ testtools>=2.2.0 # MIT
 stestr>=1.1.0 # Apache-2.0
 doc8>=0.8.1 # Apache-2.0
 astroid==1.6.5 # LGPLv2.1
-pylint==1.9.2 # GPLv2
 oslotest>=3.2.0 # Apache-2.0
 tenacity>=4.9.0  # Apache-2.0
 # Docs building

tox.ini

@@ -108,11 +108,6 @@ commands = oslo-config-generator --config-file=tools/trove-config-generator.conf
 [testenv:genpolicy]
 commands = oslopolicy-sample-generator --config-file=tools/trove-policy-generator.conf
 
-[testenv:pylint]
-deps = -r{toxinidir}/requirements.txt
-       -r{toxinidir}/test-requirements.txt
-commands = python tools/trove-pylint.py {posargs:check}
-
 [testenv:docs]
 commands = sphinx-build -W -b html doc/source doc/build/html
 

trove/db/sqlalchemy/session.py

@@ -110,10 +110,10 @@ def clean_db():
     meta.bind = engine
     meta.reflect(bind=engine)
     with contextlib.closing(engine.connect()) as con:
-        trans = con.begin()  # pylint: disable=E1101
+        trans = con.begin()
         for table in reversed(meta.sorted_tables):
             if table.name != "migrate_version":
-                con.execute(table.delete())  # pylint: disable=E1101
+                con.execute(table.delete())
         trans.commit()