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.

Sign up for GitHub

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

Jump to bottom

Add Parameter Specification to run CLI command #1551

Closed
joshmeek opened this issue Sep 25, 2019 · 0 comments · Fixed by #1582
Closed

Add Parameter Specification to run CLI command #1551

joshmeek opened this issue Sep 25, 2019 · 0 comments · Fixed by #1582
Assignees
@joshmeek
Labels
cli Related to the Prefect CLI enhancement An improvement of an existing feature

Comments

@joshmeek
Copy link

joshmeek commented Sep 25, 2019 •
edited
Loading

Current behavior

Currently the CLI command doesn't allow for the specification of parameters for a flow run

Proposed behavior

Couple options we could take here:

Multiple --parameter NAME=VALUE options (e.g. each --parameter similar to how docker build does --build-arg)

Single --parameters {name: value} that takes a dictionary

Also we should have a file option where you can provide a json file
@joshmeek joshmeek added enhancement An improvement of an existing feature cli Related to the Prefect CLI labels Sep 25, 2019
@joshmeek joshmeek self-assigned this Sep 25, 2019
@joshmeek joshmeek mentioned this issue Oct 1, 2019
Merged
3 tasks
zanieb added a commit that referenced this issue May 12, 2022
@zanieb
Merge pull request #1551 from PrefectHQ/warning-error
bb64442 
Fail tests on unhandled warnings during pytest runs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joshmeek joshmeek

Labels
cli Related to the Prefect CLI enhancement An improvement of an existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Parameter File Option to run cloud CLI command PrefectHQ/prefect
1 participant
@joshmeek