-
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
Data Engineers + dbt 1.5: Evolving the Craft for Scale #3221
Conversation
✅ Deploy Preview for docs-getdbt-com ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
||
authors: [sung_chung, kira_furuichi] | ||
|
||
tags: [data ecosystem] |
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.
think we should change the tag to either dbt tutorials
or analytics craft
, dbt product updates
!
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.
Let's go with dbt product updates!
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.
Tried out all the code examples with dbt-snowflake, and they all worked ✅
Left a handful of suggestions. This is the only blocking suggestion -- the rest are optional:
Two nits:
- The first example used
integer
whereas the second usedint
; this is fine because one is an alias of the other in Snowflake - The columns weren't the same between the first example and the second, so it was a little more juggling to follow along with the example
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
I fixed the two nits! |
Co-authored-by: Kira Furuichi <[email protected]>
Co-authored-by: Kira Furuichi <[email protected]>
…/docs.getdbt.com into blog/evolving-the-craft
@KiraFuruichi @dbeatty10 can you approve so I can officially merge? I made a couple more stylistic edits, spelling updates, and removed the bullet on Doug's 1.5 blog. |
What are you changing in this pull request and why?
Adding a new devhub blog post to pair with with the dbt 1.5 release. This adds a human, intimate touch to how dbt can best serve data engineers. More conceptual.
Checklist
Adding new pages (delete if not applicable):
website/sidebars.js
@KiraFuruichi do I need to do this? I didn't notice any of the blog content requiring this step. Edit: Not required