Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Housekeeping] [flytekit] Move input parsing to importable package #3124

Closed
2 tasks done
wild-endeavor opened this issue Nov 30, 2022 · 1 comment
Closed
2 tasks done
Labels
enhancement New feature or request flytekit FlyteKit Python related issue
Milestone

Comments

@wild-endeavor
Copy link
Contributor

wild-endeavor commented Nov 30, 2022

Task

Small cleanup task. The literal converter that works with click parsing would be useful to have separately. Move this out to a different folder so we can use it in other parts of the codebase.

Related to #208

Misc

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@wild-endeavor wild-endeavor added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Nov 30, 2022
@wild-endeavor wild-endeavor changed the title [Core feature] [flytekit] Move input parsing [Housekeeping] [flytekit] Move input parsing to importable package Nov 30, 2022
@wild-endeavor wild-endeavor added flytekit FlyteKit Python related issue Epic: Signaling Signal support, gate nodes, and HITL workflows labels Dec 1, 2022
@cosmicBboy cosmicBboy added this to the 1.4.0 milestone Jan 29, 2023
@cosmicBboy
Copy link
Contributor

Move the literal converter out of the click modules and expose this as a flytekit module

@cosmicBboy cosmicBboy modified the milestones: 1.4.0, 1.5.0 Feb 27, 2023
@cosmicBboy cosmicBboy removed the Epic: Signaling Signal support, gate nodes, and HITL workflows label Mar 10, 2023
@eapolinario eapolinario modified the milestones: 1.5.0, 2023 Q1 Backlog Mar 13, 2023
@pingsutw pingsutw removed the untriaged This issues has not yet been looked at by the Maintainers label Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request flytekit FlyteKit Python related issue
Projects
None yet
Development

No branches or pull requests

4 participants