robotools · benkiel · Aug 15, 2024 · Aug 15, 2024 · Aug 15, 2024
diff --git a/.github/workflows/black.yml b/.github/workflows/black.yml
@@ -0,0 +1,10 @@
+name: Lint
+
+on: [push, pull_request]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: psf/black@stable
diff --git a/Lib/fontParts/__init__.py b/Lib/fontParts/__init__.py
@@ -3,6 +3,7 @@
 except ImportError:
     try:
         from setuptools_scm import get_version
+
         __version__ = get_version()
     except ImportError:
-        __version__ = 'unknown'
+        __version__ = "unknown"
diff --git a/Lib/fontParts/base/anchor.py b/Lib/fontParts/base/anchor.py
@@ -1,25 +1,30 @@
 from fontTools.misc import transform
 from fontParts.base import normalizers
 from fontParts.base.base import (
-    BaseObject, TransformationMixin, InterpolationMixin, SelectionMixin,
-    PointPositionMixin, IdentifierMixin, dynamicProperty, reference
+    BaseObject,
+    TransformationMixin,
+    InterpolationMixin,
+    SelectionMixin,
+    PointPositionMixin,
+    IdentifierMixin,
+    dynamicProperty,
+    reference,
 )
 from fontParts.base.compatibility import AnchorCompatibilityReporter
 from fontParts.base.color import Color
 from fontParts.base.deprecated import DeprecatedAnchor, RemovedAnchor
 
 
 class BaseAnchor(
-                 BaseObject,
-                 TransformationMixin,
-                 DeprecatedAnchor,
-                 RemovedAnchor,
-                 PointPositionMixin,
-                 InterpolationMixin,
-                 SelectionMixin,
-                 IdentifierMixin
-                 ):
-
+    BaseObject,
+    TransformationMixin,
+    DeprecatedAnchor,
+    RemovedAnchor,
+    PointPositionMixin,
+    InterpolationMixin,
+    SelectionMixin,
+    IdentifierMixin,
+):
     """
     An anchor object. This object is almost always
     created with :meth:`BaseGlyph.appendAnchor`.
@@ -42,12 +47,7 @@ def _reprContents(self):
     # Copy
     # ----
 
-    copyAttributes = (
-        "x",
-        "y",
-        "name",
-        "color"
-    )
+    copyAttributes = ("x", "y", "name", "color")
 
     # -------
     # Parents
@@ -104,7 +104,7 @@ def _get_font(self):
             >>> anchor.x
             100
             >>> anchor.x = 101
-        """
+        """,
     )
 
     def _get_base_x(self):
@@ -147,7 +147,7 @@ def _set_x(self, value):
             >>> anchor.y
             100
             >>> anchor.y = 101
-        """
+        """,
     )
 
     def _get_base_y(self):
@@ -194,7 +194,7 @@ def _set_y(self, value):
 
             >>> anchor.index
             0
-        """
+        """,
     )
 
     def _get_base_index(self):
@@ -225,7 +225,7 @@ def _get_index(self):
             >>> anchor.name
             'my anchor'
             >>> anchor.name = None
-        """
+        """,
     )
 
     def _get_base_name(self):
@@ -272,7 +272,7 @@ def _set_name(self, value):
             >>> anchor.color
             None
             >>> anchor.color = (1, 0, 0, 0.5)
-        """
+        """,
     )
 
     def _get_base_color(self):

diff --git a/Lib/fontParts/base/bPoint.py b/Lib/fontParts/base/bPoint.py
@@ -5,21 +5,21 @@
     SelectionMixin,
     IdentifierMixin,
     dynamicProperty,
-    reference
+    reference,
 )
 from fontParts.base.errors import FontPartsError
 from fontParts.base import normalizers
 from fontParts.base.deprecated import DeprecatedBPoint, RemovedBPoint
 
 
 class BaseBPoint(
-                 BaseObject,
-                 TransformationMixin,
-                 SelectionMixin,
-                 DeprecatedBPoint,
-                 IdentifierMixin,
-                 RemovedBPoint
-                 ):
+    BaseObject,
+    TransformationMixin,
+    SelectionMixin,
+    DeprecatedBPoint,
+    IdentifierMixin,
+    RemovedBPoint,
+):
 
     def _reprContents(self):
         contents = [
@@ -197,9 +197,9 @@ def _set_bcpIn(self, value):
         x, y = absoluteBCPIn(self.anchor, value)
         segment = self._segment
         if segment.type == "move" and value != (0, 0):
-            raise FontPartsError(("Cannot set the bcpIn for the first "
-                                  "point in an open contour.")
-                                 )
+            raise FontPartsError(
+                ("Cannot set the bcpIn for the first " "point in an open contour.")
+            )
         else:
             offCurves = segment.offCurve
             if offCurves:
@@ -251,9 +251,9 @@ def _set_bcpOut(self, value):
         segment = self._segment
         nextSegment = self._nextSegment
         if nextSegment.type == "move" and value != (0, 0):
-            raise FontPartsError(("Cannot set the bcpOut for the last "
-                                  "point in an open contour.")
-                                 )
+            raise FontPartsError(
+                ("Cannot set the bcpOut for the last " "point in an open contour.")
+            )
         else:
             offCurves = nextSegment.offCurve
             if offCurves:
@@ -304,8 +304,7 @@ def _get_type(self):
             bType = "corner"
 
         if bType is None:
-            raise FontPartsError("A %s point can not be converted to a bPoint."
-                                         % typ)
+            raise FontPartsError("A %s point can not be converted to a bPoint." % typ)
         return bType
 
     def _set_type(self, value):
@@ -331,11 +330,14 @@ def _set_type(self, value):
     # Identification
     # --------------
 
-    index = dynamicProperty("index",
-                            ("The index of the bPoint within the ordered "
-                             "list of the parent contour's bPoints. None "
-                             "if the bPoint does not belong to a contour.")
-                            )
+    index = dynamicProperty(
+        "index",
+        (
+            "The index of the bPoint within the ordered "
+            "list of the parent contour's bPoints. None "
+            "if the bPoint does not belong to a contour."
+        ),
+    )
 
     def _get_base_index(self):
         if self.contour is None:
@@ -381,14 +383,20 @@ def round(self):
         Round coordinates.
         """
         x, y = self.anchor
-        self.anchor = (normalizers.normalizeVisualRounding(x),
-                       normalizers.normalizeVisualRounding(y))
+        self.anchor = (
+            normalizers.normalizeVisualRounding(x),
+            normalizers.normalizeVisualRounding(y),
+        )
         x, y = self.bcpIn
-        self.bcpIn = (normalizers.normalizeVisualRounding(x),
-                      normalizers.normalizeVisualRounding(y))
+        self.bcpIn = (
+            normalizers.normalizeVisualRounding(x),
+            normalizers.normalizeVisualRounding(y),
+        )
         x, y = self.bcpOut
-        self.bcpOut = (normalizers.normalizeVisualRounding(x),
-                       normalizers.normalizeVisualRounding(y))
+        self.bcpOut = (
+            normalizers.normalizeVisualRounding(x),
+            normalizers.normalizeVisualRounding(y),
+        )
 
 
 def relativeBCPIn(anchor, BCPIn):

diff --git a/Lib/fontParts/base/base.py b/Lib/fontParts/base/base.py
@@ -9,8 +9,8 @@
 # Helpers
 # -------
 
-class dynamicProperty(object):
 
+class dynamicProperty(object):
     """
     This implements functionality that is very similar
     to Python's built in property function, but makes
@@ -111,6 +111,7 @@ def interpolate(a, b, v):
 # Base Objects
 # ------------
 
+
 class BaseObject(object):
 
     # --------------
@@ -138,9 +139,7 @@ def __repr__(self):
         else:
             contents = ""
         s = "<{className}{contents} at {address}>".format(
-            className=self.__class__.__name__,
-            contents=contents,
-            address=id(self)
+            className=self.__class__.__name__, contents=contents, address=id(self)
         )
         return s
 
@@ -228,8 +227,9 @@ def raiseNotImplementedError(self):
         the base classes. So, it's here for convenience.
         """
         raise NotImplementedError(
-            "The {className} subclass does not implement this method."
-            .format(className=self.__class__.__name__)
+            "The {className} subclass does not implement this method.".format(
+                className=self.__class__.__name__
+            )
         )
 
     # ---------------------
@@ -290,8 +290,7 @@ def items(self):
         items = self._items()
         if self.keyNormalizer is not None and self.valueNormalizer is not None:
             values = [
-                (self.keyNormalizer.__func__(key),
-                 self.valueNormalizer.__func__(value))
+                (self.keyNormalizer.__func__(key), self.valueNormalizer.__func__(value))
                 for (key, value) in items
             ]
         return values
@@ -654,7 +653,8 @@ def isCompatible(self, other, cls):
             raise TypeError(
                 """Compatibility between an instance of %r and an \
                 instance of %r can not be checked."""
-                % (cls.__name__, other.__class__.__name__))
+                % (cls.__name__, other.__class__.__name__)
+            )
         reporter = self.compatibilityReporterClass(self, other)
         self._isCompatible(other, reporter)
         return not reporter.fatal, reporter
@@ -680,7 +680,7 @@ class SelectionMixin(object):
             >>> obj.selected
             False
             >>> obj.selected = True
-        """
+        """,
     )
 
     def _get_base_selected(self):
@@ -733,7 +733,6 @@ def _setSelectedSubObjects(cls, subObjects, selected):
 
 
 class PointPositionMixin(object):
-
     """
     This adds a ``position`` attribute as a dyanmicProperty,
     for use as a mixin with objects that have ``x`` and ``y``
@@ -784,7 +783,7 @@ class IdentifierMixin(object):
 
         To request an identifier if it does not exist use
         `object.getIdentifier()`
-        """
+        """,
     )
 
     def _get_base_identifier(self):
@@ -834,6 +833,7 @@ def reference(obj):
     # return weakref.ref(obj)
     def wrapper():
         return obj
+
     return wrapper
 
 

diff --git a/Lib/fontParts/base/color.py b/Lib/fontParts/base/color.py
@@ -1,29 +1,24 @@
 class Color(tuple):
-
     """
     An color object. This follows the :ref:`type-color`.
     """
 
     def _get_r(self):
         return self[0]
 
-    r = property(_get_r,
-                 "The color's red component as :ref:`type-int-float`.")
+    r = property(_get_r, "The color's red component as :ref:`type-int-float`.")
 
     def _get_g(self):
         return self[1]
 
-    g = property(_get_g,
-                 "The color's green component as :ref:`type-int-float`.")
+    g = property(_get_g, "The color's green component as :ref:`type-int-float`.")
 
     def _get_b(self):
         return self[2]
 
-    b = property(_get_b,
-                 "The color's blue component as :ref:`type-int-float`.")
+    b = property(_get_b, "The color's blue component as :ref:`type-int-float`.")
 
     def _get_a(self):
         return self[3]
 
-    a = property(_get_a,
-                 "The color's alpha component as :ref:`type-int-float`.")
+    a = property(_get_a, "The color's alpha component as :ref:`type-int-float`.")