Skip to content

Commit 6866bba

Browse files
committed
fix: use ConfigMetadata defaults in CLI options
- Import ConfigMetadata class - Use DEFAULT_DESCRIPTION for description option - Use DEFAULT_REQUIRED for required option - Use DEFAULT_SENSITIVE for sensitive option
1 parent c7a710d commit 6866bba

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

helm_values_manager/cli.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
from helm_values_manager.commands.add_value_config_command import AddValueConfigCommand
77
from helm_values_manager.commands.init_command import InitCommand
88
from helm_values_manager.commands.set_value_command import SetValueCommand
9+
from helm_values_manager.models.config_metadata import ConfigMetadata
910
from helm_values_manager.utils.logger import HelmLogger
1011

1112
COMMAND_INFO = "helm values-manager"
@@ -48,10 +49,14 @@ def init(
4849
@app.command("add-value-config")
4950
def add_value_config(
5051
path: str = typer.Option(..., "--path", "-p", help="Configuration path (e.g., 'app.replicas')"),
51-
description: str = typer.Option("", "--description", "-d", help="Description of what this configuration does"),
52-
required: bool = typer.Option(False, "--required", "-r", help="Whether this configuration is required"),
52+
description: str = typer.Option(
53+
ConfigMetadata.DEFAULT_DESCRIPTION, "--description", "-d", help="Description of what this configuration does"
54+
),
55+
required: bool = typer.Option(
56+
ConfigMetadata.DEFAULT_REQUIRED, "--required", "-r", help="Whether this configuration is required"
57+
),
5358
sensitive: bool = typer.Option(
54-
False,
59+
ConfigMetadata.DEFAULT_SENSITIVE,
5560
"--sensitive",
5661
"-s",
5762
help="Whether this configuration contains sensitive data (coming in v0.2.0)",

0 commit comments

Comments
 (0)