linting & pin docformatter

- fix for 'pylint: disable'

- pin docformatter, As it seems the bug that @avylove created may not
  been addressed, freeze docformatter for now, to prevent failing CI

  PyCQA/docformatter#264

blessed/keyboard.py

@@ -455,8 +455,11 @@ def _read_until(term, pattern, timeout):
 #: 350ms, or 0.35 seconds. It is still a bit conservative, for remote telnet or
 #: ssh servers, for example.
 DEFAULT_ESCDELAY = 0.35
+
+
 def _reinit_escdelay():
-    # pylint: disable=W0603 Using the global statement, this is necessary to
+    # pylint: disable=W0603
+    # Using the global statement: this is necessary to
     # allow test coverage without complex module reload
     global DEFAULT_ESCDELAY
     if os.environ.get('ESCDELAY'):
@@ -465,6 +468,8 @@ def _reinit_escdelay():
         except ValueError:
             # invalid values of 'ESCDELAY' are ignored
             pass
+
+
 _reinit_escdelay()
 
 

blessed/terminal.py

@@ -18,13 +18,13 @@
 
 # local
 from .color import COLOR_DISTANCE_ALGORITHMS
-from .keyboard import (_time_left,
+from .keyboard import (DEFAULT_ESCDELAY,
+                       _time_left,
                        _read_until,
                        resolve_sequence,
                        get_keyboard_codes,
                        get_leading_prefixes,
-                       get_keyboard_sequences,
-                       DEFAULT_ESCDELAY)
+                       get_keyboard_sequences)
 from .sequences import Termcap, Sequence, SequenceTextWrapper
 from .colorspace import RGB_256TABLE
 from .formatters import (COLORS,

tests/accessories.py

@@ -61,7 +61,7 @@ class as_subprocess(object):  # pylint: disable=too-few-public-methods
     def __init__(self, func):
         self.func = func
 
-    def __call__(self, *args, **kwargs):  # pylint: disable=too-many-locals, too-complex
+    def __call__(self, *args, **kwargs):  # pylint: disable=too-many-locals,too-complex
         if IS_WINDOWS:
             self.func(*args, **kwargs)
             return

tests/test_keyboard.py

@@ -365,6 +365,7 @@ def child(kind):  # pylint: disable=too-many-statements
 
     child('xterm')
 
+
 def test_ESCDELAY_unset_unchanged():
     """Unset ESCDELAY leaves DEFAULT_ESCDELAY unchanged in _reinit_escdelay()."""
     if 'ESCDELAY' in os.environ:
@@ -374,6 +375,7 @@ def test_ESCDELAY_unset_unchanged():
     blessed.keyboard._reinit_escdelay()
     assert blessed.keyboard.DEFAULT_ESCDELAY == prev_value
 
+
 def test_ESCDELAY_bad_value_unchanged():
     """Invalid ESCDELAY leaves DEFAULT_ESCDELAY unchanged in _reinit_escdelay()."""
     os.environ['ESCDELAY'] = 'XYZ123!'
@@ -383,6 +385,7 @@ def test_ESCDELAY_bad_value_unchanged():
     assert blessed.keyboard.DEFAULT_ESCDELAY == prev_value
     del os.environ['ESCDELAY']
 
+
 def test_ESCDELAY_10ms():
     """Verify ESCDELAY modifies DEFAULT_ESCDELAY in _reinit_escdelay()."""
     os.environ['ESCDELAY'] = '1234'

tox.ini

@@ -67,8 +67,9 @@ commands =
     autopep8 --in-place  --recursive --aggressive --aggressive blessed/ bin/ setup.py
 
 [testenv:docformatter]
+# docformatter pinned due to https://github.com/PyCQA/docformatter/issues/264
 deps =
-    docformatter
+    docformatter<1.7.4
     untokenize
 commands =
     docformatter \
@@ -83,8 +84,9 @@ commands =
         {toxinidir}/docs/conf.py
 
 [testenv:docformatter_check]
+# docformatter pinned due to https://github.com/PyCQA/docformatter/issues/264
 deps =
-    docformatter
+    docformatter<1.7.4
     untokenize
 commands =
     docformatter \