Merge pull request #166 from edx/arbrandes/fix-empty-yaml

fix: properly deal with empty yaml files
openedx · Apr 9, 2021 · f4e7a8d · f4e7a8d
2 parents c934c4f + 86f7e90
commit f4e7a8d
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/edx_repo_tools/data.py b/edx_repo_tools/data.py
@@ -62,9 +62,13 @@ def iter_openedx_yaml(hub, orgs, branches=None):
             if contents is not None:
                 LOGGER.debug("Found openedx.yaml at %s:%s", repo.full_name, branch)
                 try:
-                    yield repo, yaml.safe_load(contents.decoded)
+                    data = yaml.safe_load(contents.decoded)
                 except Exception as exc:
                     LOGGER.error("Couldn't parse openedx.yaml from %s:%s, skipping repo", repo.full_name, branch, exc_info=True)
+                else:
+                    if data is not None:
+                        yield repo, data
+
                 break