Merge pull request #21 from shreypandey/main

Updated logic for not including existing config vars
microsoft · Sep 9, 2024 · a2b26d7 · a2b26d7
2 parents 7f8d24b + e5a6ba6
commit a2b26d7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/nl2dsl/__init__.py b/nl2dsl/__init__.py
@@ -181,8 +181,9 @@ def add_azure_keys(self, dsl):
         "FAST_MODEL",
         "SLOW_MODEL",
         ]
+        existing_keys = set(config_var["name"] for config_var in dsl["config_vars"])
         for key in AZURE_CREDENTIALS:
-            if key not in dsl["config_vars"]:
+            if key not in existing_keys:
                 dsl["config_vars"].append({"name": key, "value": ""})
         return dsl