re-enabling many linters checks

setup.py

@@ -16,7 +16,6 @@
 # limitations under the License.
 #
 # See AUTHORS.txt for the list of Authors and LICENSE.txt for the License.
-import os
 from setuptools import setup
 
 # Read the contents of the README file
@@ -38,11 +37,11 @@ setup(
     package_dir={'': 'Lib'},
     packages=['gflanguages'],
     package_data={'gflanguages': [
-                      "data/languages/*.textproto",
-                      "data/regions/*.textproto",
-                      "data/scripts/*.textproto"
-                  ]
-                 },
+                     "data/languages/*.textproto",
+                     "data/regions/*.textproto",
+                     "data/scripts/*.textproto"
+                 ]
+    },
     zip_safe=False,
     classifiers=[
         'Intended Audience :: Developers',

tests/test_gflanguages_api.py

@@ -1,7 +1,7 @@
 import os
-import pytest
 from gflanguages import lang_support
 
+
 def portable_path(p):
     return os.path.join(*p.split('/'))
 

tox.ini

@@ -60,65 +60,6 @@ exclude =
 # The Sphinx configuration file is mostly autogenerated, ignore it
     docs/source/conf.py,
 
-
-# The ignore-list below comes from FontBakery linting setup.
-# The `gflanguages` module has a much simples codebase, so we may be in much better shape in here.
-# So, perhaps, we could safely enable many of these lint-checks.
 max-line-length=100
 ignore = 
-    # ITEMS BELOW ARE MEDIUM PRIORITY TO FIX AND REMOVE FROM IGNORE LIST
-    E402, # module level import not at top of file
-    E711, # comparison to None should be 'if cond is None
-    E712, # comparison to True should be 'if cond is True:' or 'if cond:'
-    E713, # test for membership should be 'not in'
-    E722, # do not use bare 'except'
-    E741, # ambiguous variable name
-    F401, # '<module>' imported but unused
-    F403, # 'from <module> import *' used; unable to detect undefined names
-    F405, # '<name>' may be undefined, or defined from star imports
-
-    # ITEMS BELOW ARE LOWER PRIORITY TO FIX AND REMOVE FROM IGNORE LIST
-    E111, # indentation is not a multiple of four
-    E114, # indentation is not a multiple of four (comment)
-    E116, # unexpected indentation (comment)
-    E117, # over-indented (comment)
-    E122, # continuation line missing indentation or outdented
-    E123, # closing bracket does not match indentation of opening bracket's line
-    E124, # closing bracket does not match visual indentation
-    E125, # continuation line with same indent as next logical line
     E126, # continuation line over-indented for hanging indent
-    E127, # continuation line over-indented for visual indent
-    E128, # continuation line under-indented for visual indent
-    E129, # visually indented line with same indent as next logical line
-    E131, # continuation line unaligned for hanging indent
-    E201, # whitespace after '{'
-    E202, # whitespace before ')'
-    E203, # whitespace before ','
-    E211, # whitespace before '('
-    E221, # multiple spaces before operator
-    E222, # multiple spaces after operator
-    E225, # missing whitespace around operator
-    E226, # missing whitespace around arithmetic operator
-    E231, # missing whitespace after ','
-    E241, # multiple spaces after ':'
-    E251, # unexpected spaces around keyword / parameter equals
-    E261, # at least two spaces before inline comment
-    E262, # inline comment should start with '# '
-    E265, # block comment should start with '# '
-    E266, # too many leading '#' for block comment
-    E272, # multiple spaces before keyword
-    E302, # expected <n> blank lines, found <m>
-    E303, # too many blank lines (<n>)
-    E305, # expected 2 blank lines after class or function definition, found <n>
-    E306, # expected 1 blank line before a nested definition, found <n>
-    E502, # the backslash is redundant between brackets
-    E701, # multiple statements on one line (colon)
-    E704, # multiple statements on one line (def)
-    E731, # do not assign a lambda expression, use a def
-    F541, # f-string is missing placeholders
-    W191, # indentation contains tabs
-    W291, # trailing whitespace
-    W293, # blank line contains whitespace
-    W391, # blank line at end of file
-    W503, # line break before binary operator
-    W504, # line break after binary operator