Fix parsing of outcomes

mozilla · Nov 24, 2022 · e53d82a · e53d82a
1 parent 932c0d1
commit e53d82a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/metric_config_parser/config.py b/metric_config_parser/config.py
@@ -381,13 +381,13 @@ def from_github_repos(
             return ConfigCollection.from_github_repo()
 
         configs = None
+
         for repo in repo_urls:
             if configs is None:
                 configs = ConfigCollection.from_github_repo(repo, is_private=is_private)
             else:
                 collection = ConfigCollection.from_github_repo(repo, is_private=is_private)
                 configs.merge(collection)
-
         return configs or ConfigCollection.from_github_repo()
 
     def spec_for_outcome(self, slug: str, platform: str) -> Optional[OutcomeSpec]:
@@ -514,6 +514,8 @@ def merge(self, other: "ConfigCollection"):
             if outcome.slug not in slugs:
                 outcomes.append(outcome)
 
+        self.outcomes = outcomes
+
         # merge definitions
         other_definitions = {
             definition.slug: definition for definition in deepcopy(other.definitions)

diff --git a/setup.py b/setup.py
@@ -63,5 +63,5 @@ def text_from_file(path):
         [console_scripts]
         metric-config-parser=metric_config_parser.cli:cli
     """,
-    version="2022.11.2",
+    version="2022.11.3",
 )