Skip to content

Commit 7936170

Browse files
committed
feat: Support ConfigManager overrides without specifying set_value
No longer requires the `set_value` parameter. Simplifies usage whenever you want to allow environment variables to take precedence over the config database and you don't have any value to set. Previously, you had to manually pass `None` for that scenario. Now, the `None` is implicit if the final parameter is omitted.
1 parent f0976f2 commit 7936170

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

comfy_cli/config_manager.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ def get(self, key):
4343
"""
4444
return self.config["DEFAULT"].get(key, None) # Returns None if the key does not exist
4545

46-
def get_or_override(self, env_key: str, config_key: str, set_value: Optional[str]) -> str | None:
46+
def get_or_override(self, env_key: str, config_key: str, set_value: Optional[str] = None) -> str | None:
4747
"""
4848
Resolves and conditionally stores a config value.
4949

0 commit comments

Comments
 (0)