-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
asset list command #12790
asset list command #12790
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
@@ -81,6 +81,34 @@ def execute_materialize_command(instance: DagsterInstance, kwargs: Mapping[str, | |||
) | |||
|
|||
|
|||
@asset_cli.command(name="list", help="List assets") | |||
@python_origin_target_argument | |||
@click.option("--select", help="Asset selection to target", required=False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Asset selection" is a bit of a fraught term but I think it works here. Should we at least add some examples (e.g. "prefixed/asset*")?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to address this in a followup after adding asset selection syntax docs that we can link to: #12871
Summary & Motivation
How I Tested These Changes