Review comments

Author: spoorcc

dfetch/manifest/project.py

@@ -278,7 +278,7 @@
 
 from dfetch.manifest.remote import Remote
 from dfetch.manifest.version import Version
-from dfetch.util.util import always_str_list
+from dfetch.util.util import always_str_list, str_if_possible
 
 ProjectEntryDict = TypedDict(
     "ProjectEntryDict",
@@ -468,7 +468,7 @@ def as_yaml(self) -> dict[str, Union[str, list[str]]]:
             "src": self._src,
             "dst": self._dst if self._dst != self._name else None,
             "url": self._url,
-            "patch": self._patch,
+            "patch": str_if_possible(self._patch),
             "branch": self._branch,
             "tag": self._tag,
             "repo-path": self._repo_path,

dfetch/project/subproject.py

@@ -132,7 +132,7 @@ def update(
         for patch in self.__project.patch:
             if os.path.exists(patch):
                 self.apply_patch(patch)
-                applied_patches += [patch]
+                applied_patches.append(patch)
             else:
                 logger.warning(f"Skipping non-existent patch {patch}")