style: PR feedback

Author: yt-ms

src/structurizr/model/component.py

@@ -82,7 +82,7 @@ class Component(StaticStructureElement):
     def __init__(
         self,
         *,
-        parent: "Container" = None,
+        parent: Optional["Container"] = None,
         technology: str = "",
         code_elements: Iterable[CodeElement] = (),
         size: Optional[int] = None,
@@ -109,7 +109,7 @@ def __init__(
 
     @classmethod
     def hydrate(cls, component_io: ComponentIO, container: "Container") -> "Component":
-        """"""
+        """Create and hydrate a new `Component` instance from its IO."""
         return cls(
             **cls.hydrate_arguments(component_io),
             parent=container,

src/structurizr/model/container.py

@@ -79,7 +79,7 @@ class Container(StaticStructureElement):
     def __init__(
         self,
         *,
-        parent: "SoftwareSystem" = None,
+        parent: Optional["SoftwareSystem"] = None,
         technology: str = "",
         components: Iterable[Component] = (),
         **kwargs,
@@ -105,7 +105,7 @@ def __init__(
     @property
     def components(self) -> Iterable[Component]:
         """Return read-only list of child components."""
-        return [c for c in self._components]
+        return list(self._components)
 
     @classmethod
     def hydrate(
@@ -144,7 +144,7 @@ def __iadd__(self, component: Component) -> "Container":
 
         if self.get_component_with_name(component.name):
             raise ValueError(
-                f"Component with name {component.name} already exists for {self}."
+                f"Component with name {component.name} already exists in {self}."
             )
 
         if component.parent is None: