Revert "Also copy Literals outside of experimental feature"

This reverts commit 3ee9ef3.

Signed-off-by: Alice Purcell <[email protected]>

src/hera/shared/_type_util.py

@@ -83,13 +83,6 @@ def get_workflow_annotation(annotation: Any) -> Optional[Union[Artifact, Paramet
     return metadata[0]
 
 
-def add_metadata_from_type(parameter: Parameter, annotation: Any) -> None:
-    if not parameter.enum:
-        type_ = unwrap_annotation(annotation)
-        if get_origin(type_) is Literal:
-            parameter.enum = list(get_args(type_))
-
-
 def construct_io_from_annotation(python_name: str, annotation: Any) -> Union[Parameter, Artifact]:
     """Constructs a Parameter or Artifact object based on annotations.
 

src/hera/workflows/script.py

@@ -48,7 +48,6 @@
 )
 from hera.shared._pydantic import _PYDANTIC_VERSION, root_validator, validator
 from hera.shared._type_util import (
-    add_metadata_from_type,
     construct_io_from_annotation,
     get_workflow_annotation,
     is_subscripted,
@@ -380,7 +379,6 @@ def _get_parameters_from_callable(source: Callable) -> List[Parameter]:
             default = MISSING
 
         param = Parameter(name=p.name, default=default)
-        add_metadata_from_type(param, p.annotation)
         parameters.append(param)
 
     return parameters

tests/test_script_annotations.py

@@ -486,12 +486,10 @@ def test_script_with_param(global_config_fixture, module_name):
         pytest.param("tests.script_annotations.pydantic_io_literals", "my_str", id="pydantic-io"),
     ],
 )
-@pytest.mark.parametrize("experimental_feature", ["", "script_annotations", "script_pydantic_io"])
-def test_script_literals(global_config_fixture, module_name, input_name, experimental_feature):
+def test_script_literals(global_config_fixture, module_name, input_name):
     """Test that Literals work correctly as direct type annotations."""
     # GIVEN
-    if experimental_feature:
-        global_config_fixture.experimental_features[experimental_feature] = True
+    global_config_fixture.experimental_features["script_annotations"] = True
 
     # Force a reload of the test module, as the runner performs "importlib.import_module", which
     # may fetch a cached version