#1706 Remove ensure literal
Pull request opened by
wild-endeavor
TL;DR
The ensure_literal call in workflow.py was a duplicate that we just didn't realize was a duplicate. Removing and updating the existing function in promise with the correct union handling behavior.
Type
☑︎ Bug Fix
☐ Feature
☐ Plugin
Are all requirements met?
☑︎ Code completed
☑︎ Smoke tested
☐ Unit tests added
☐ Code documentation added
☐ Any pending items have an associated Issue
Complete description
This was discovered while implementing the now-defunct
#1700
Tracking Issue
NA
flyteorg/flytekit
GitHub Actions: build-plugins (3.11, flytekit-sqlalchemy)
GitHub Actions: build-plugins (3.11, flytekit-snowflake)
GitHub Actions: build-plugins (3.11, flytekit-polars)
GitHub Actions: build-plugins (3.11, flytekit-papermill)
GitHub Actions: build-plugins (3.11, flytekit-pandera)
GitHub Actions: build-plugins (3.11, flytekit-kf-tensorflow)
GitHub Actions: build-plugins (3.11, flytekit-envd)
GitHub Actions: build-plugins (3.10, flytekit-vaex)
✅ 22 other checks have passed
22/30 successful checks