-
Notifications
You must be signed in to change notification settings - Fork 982
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
Adding semantic models to list #4156
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
@@ -10,7 +10,7 @@ The `dbt ls` command lists resources in your dbt project. It accepts selector ar | |||
### Usage | |||
``` | |||
dbt ls | |||
[--resource-type {model,source,seed,snapshot,metric,test,exposure,analysis,default,all}] | |||
[--resource-type {model,semantic_model,source,seed,snapshot,metric,test,exposure,analysis,default,all}] |
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.
This doesn't have a versioning component for the metric
and semantic_model
types, but maybe that is okay? 🤷
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.
One non-blocking comment within website/docs/reference/commands/list.md
about versioning components. Not sure if we want to leave that portion as unversioned or not 🤷
Everything else looks good!
Previews:
What are you changing in this pull request and why?
Adding semantic model support to
list
for dbt v1.6+Resolves: #4134
Checklist
Add versioning components, as described in Versioning Docs
Add a note to the prerelease version Migration Guide
Review the Content style guide and About versioning so my content adheres to these guidelines.
Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."