Enhance label description checks in ChangeClass and Change classes

Updated the label description checks to ensure they are not None before performing regex matching. This improves robustness in handling labels in the changelog processing.

Author: jmcouffin

dev/_changelog.py

@@ -26,7 +26,7 @@ def __init__(self, label: github.LabelInfo) -> None:
         self._label = label
         self._aspect_type = None
         self._aspect_pattern = ChangeClass.DefaultPattern
-        if m := re.match(r".*\[(.+?)(->(.+?))?\].*", label.description):
+        if label.description and (m := re.match(r".*\[(.+?)(->(.+?))?\].*", label.description)):
             self._aspect_type = m.groups()[0]
             if pattern := m.groups()[2]:
                 self._aspect_pattern = f"- {pattern}"
@@ -125,7 +125,7 @@ def subsystems(self):
             return [
                 ChangeClass(x)
                 for x in self._ticketdata.labels
-                if "[subsystem" in x.description
+                if x.description and "[subsystem" in x.description
             ]
         return []
 
@@ -136,7 +136,7 @@ def classes(self):
             return [
                 ChangeClass(x)
                 for x in self._ticketdata.labels
-                if "[class" in x.description
+                if x.description and "[class" in x.description
             ]
         return []
 

dev/scripts/github.py

@@ -49,7 +49,7 @@ def get_ticket(ticket: str):
         ticket_labels = []
         if labels := issue.get("labels", []):
             ticket_labels = [
-                LabelInfo(name=x["name"], description=x["description"])
+                LabelInfo(name=x["name"], description=x.get("description") or "")
                 for x in labels
             ]
         else: