Improve _load_required_env error message clarity
Distinguish between missing argument and missing environment variable in the RuntimeError message to speed up debugging.
This commit is contained in:
@@ -53,10 +53,14 @@ def _load_json(path: Path) -> dict[str, Any]:
|
|||||||
|
|
||||||
def _load_required_env(name: str, value: str | None) -> str:
|
def _load_required_env(name: str, value: str | None) -> str:
|
||||||
# 优先使用传入的 value,否则读取同名环境变量;两者均缺失时抛出 RuntimeError
|
# 优先使用传入的 value,否则读取同名环境变量;两者均缺失时抛出 RuntimeError
|
||||||
resolved = value or os.environ.get(name)
|
if value:
|
||||||
if not resolved:
|
return value
|
||||||
raise RuntimeError(f"Missing required value: {name}.")
|
env_value = os.environ.get(name)
|
||||||
return resolved
|
if env_value:
|
||||||
|
return env_value
|
||||||
|
raise RuntimeError(
|
||||||
|
f"Missing required value '{name}': not passed as argument and not set as environment variable."
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
def default_output_dir() -> Path:
|
def default_output_dir() -> Path:
|
||||||
|
|||||||
Reference in New Issue
Block a user