Remove and update all pylint disables

Author: lesamouraipourpre

adafruit_display_text/__init__.py

@@ -17,6 +17,8 @@
 def wrap_text_to_pixels(
     string: str, max_width: int, font=None, indent0: str = "", indent1: str = ""
 ) -> List[str]:
+    # pylint: disable=too-many-branches, too-many-locals
+
     """wrap_text_to_pixels function
     A helper that will return a list of lines with word-break wrapping.
     Leading and trailing whitespace in your string will be removed. If
@@ -34,7 +36,6 @@ def wrap_text_to_pixels(
     :rtype: List[str]
 
     """
-    # pylint: disable=too-many-locals, too-many-branches
     if font is None:
 
         def measure(text):
@@ -153,6 +154,8 @@ def chunks(lst, n):
 
 
 class LabelBase(Group):
+    # pylint: disable=too-many-instance-attributes
+
     """Superclass that all other types of labels will extend. This contains
     all of the properties and functions that work the same way in all labels.
 
@@ -187,7 +190,6 @@ class LabelBase(Group):
     :param str label_direction: string defining the label text orientation. See the
      subclass documentation for the possible values."""
 
-    # pylint: disable=unused-argument,  too-many-instance-attributes, too-many-locals, too-many-arguments
     def __init__(
         self,
         font,
@@ -208,8 +210,10 @@ def __init__(
         base_alignment: bool = False,
         tab_replacement: Tuple[int, str] = (4, " "),
         label_direction: str = "LTR",
-        **kwargs,
+        **kwargs,  # pylint: disable=unused-argument
     ) -> None:
+        # pylint: disable=too-many-arguments, too-many-locals
+
         super().__init__(x=x, y=y, scale=1)
 
         self._font = font

adafruit_display_text/bitmap_label.py

@@ -80,9 +80,6 @@ class Label(LabelBase):
      configurations possibles ``LTR``-Left-To-Right ``RTL``-Right-To-Left
      ``UPD``-Upside Down ``UPR``-Upwards ``DWR``-Downwards. It defaults to ``LTR``"""
 
-    # pylint: disable=unused-argument, too-many-instance-attributes, too-many-locals, too-many-arguments
-    # pylint: disable=too-many-branches, no-self-use, too-many-statements
-
     def __init__(self, font, save_text=True, **kwargs) -> None:
 
         super().__init__(font, **kwargs)
@@ -109,6 +106,7 @@ def _reset_text(
         line_spacing: float = None,
         scale: int = None,
     ) -> None:
+        # pylint: disable=too-many-branches, too-many-statements
 
         # Store all the instance variables
         if font is not None:
@@ -256,6 +254,8 @@ def _line_spacing_ypixels(font, line_spacing: float) -> int:
     def _text_bounding_box(
         self, text: str, font
     ) -> Tuple[int, int, int, int, int, int]:
+        # pylint: disable=too-many-locals
+
         ascender_max, descender_max = self._ascent, self._descent
 
         lines = 1
@@ -330,7 +330,6 @@ def _text_bounding_box(
             final_y_offset_loose,
         )
 
-    # pylint: disable=too-many-nested-blocks
     def _place_text(
         self,
         bitmap,
@@ -342,6 +341,8 @@ def _place_text(
         # when copying glyph bitmaps (this is important for slanted text
         # where rectangular glyph boxes overlap)
     ) -> Tuple[int, int, int, int]:
+        # pylint: disable=too-many-arguments, too-many-locals
+
         # placeText - Writes text into a bitmap at the specified location.
         #
         # Note: scale is pushed up to Group level
@@ -441,6 +442,7 @@ def _blit(
         skip_index: int = None,  # palette index that will not be copied
         # (for example: the background color of a glyph)
     ) -> None:
+        # pylint: disable=no-self-use, too-many-arguments
 
         if hasattr(bitmap, "blit"):  # if bitmap has a built-in blit function, call it
             # this function should perform its own input checks

adafruit_display_text/label.py

@@ -37,6 +37,8 @@
 
 
 class Label(LabelBase):
+    # pylint: disable=too-many-instance-attributes
+
     """A label displaying a string of text. The origin point set by ``x`` and ``y``
     properties will be the left edge of the bounding box, and in the center of a M
     glyph (if its one line), or the (number of lines * linespacing + M)/2. That is,
@@ -77,9 +79,6 @@ class Label(LabelBase):
      configurations possibles ``LTR``-Left-To-Right ``RTL``-Right-To-Left
      ``TTB``-Top-To-Bottom ``UPR``-Upwards ``DWR``-Downwards. It defaults to ``LTR``"""
 
-    # pylint: disable=too-many-instance-attributes, too-many-locals
-    # This has a lot of getters/setters, maybe it needs cleanup.
-
     def __init__(self, font, **kwargs) -> None:
         self._background_palette = Palette(1)
         self._added_background_tilegrid = False
@@ -226,10 +225,9 @@ def _set_background_color(self, new_color: int) -> None:
                 self._local_group.pop(0)
                 self._added_background_tilegrid = False
 
-    # pylint: disable = too-many-branches, too-many-statements
-    def _update_text(
-        self, new_text: str
-    ) -> None:  # pylint: disable=too-many-locals ,too-many-branches, too-many-statements
+    def _update_text(self, new_text: str) -> None:
+        # pylint: disable=too-many-branches,too-many-statements
+
         x = 0
         y = 0
         if self._added_background_tilegrid:
@@ -379,8 +377,10 @@ def _update_text(
 
         while len(self._local_group) > tilegrid_count:  # i:
             self._local_group.pop()
-        # pylint: disable=invalid-unary-operand-type
+
         if self._label_direction == "RTL":
+            # pylint: disable=invalid-unary-operand-type
+            # type-checkers think left can be None
             self._bounding_box = (-left, top, left - right, bottom - top)
         if self._label_direction == "TTB":
             self._bounding_box = (left, top, right - left, bottom - top)