Skip to content

Commit e97df57

Browse files
author
Tim Huff
committed
adjusting a comment
1 parent c30ab7b commit e97df57

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

src/groundlight/cli.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -146,10 +146,9 @@ def wrapper(*args, **kwargs):
146146
break
147147
if not found_supported_type:
148148
cli_unsupported_params.append(name)
149-
elif is_experimental and not is_cli_representable(annotation):
150-
# For experimental methods only: proactively flag non-Union types that Typer cannot
151-
# represent (e.g. dict, list, custom models) so the caller can skip them gracefully
152-
# before Typer raises a deferred RuntimeError at cli_app() invocation time.
149+
elif not is_cli_representable(annotation):
150+
# Proactively flag non-Union types that Typer cannot represent (e.g. dict, list,
151+
# custom models) before Typer raises a deferred RuntimeError at invocation time.
153152
cli_unsupported_params.append(name)
154153
# Ideally we could just not list the unsupported params, but it doesn't seem natively supported by Typer
155154
# and requires more metaprogamming than makes sense at the moment. For now, we require methods to support str

0 commit comments

Comments
 (0)