Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Pass command-line arguments #193

Merged
merged 4 commits into from
Jan 21, 2021
Merged

Pass command-line arguments #193

merged 4 commits into from
Jan 21, 2021

Conversation

yindia
Copy link
Contributor

@yindia yindia commented Oct 8, 2020

TL;DR

Pass command-line arguments

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

Tracking Issue

flyteorg/flyte#167

Follow-up issue

@yindia
Copy link
Contributor Author

yindia commented Jan 4, 2021

@kumare3 can you review the PR?

@EngHabu
Copy link
Contributor

EngHabu commented Jan 13, 2021

Hey @evalsocket, can you rebase and make sure tests are green?

…h 'master' of github.com:lyft/flytepropeller into fix-167
@yindia
Copy link
Contributor Author

yindia commented Jan 21, 2021

@EngHabu pr is ready

Copy link
Contributor

@EngHabu EngHabu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@anandswaminathan anandswaminathan merged commit 78cc1e4 into flyteorg:master Jan 21, 2021
EngHabu added a commit to EngHabu/flytepropeller that referenced this pull request Jan 26, 2021
anandswaminathan pushed a commit that referenced this pull request Jan 26, 2021
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants