-
Notifications
You must be signed in to change notification settings - Fork 999
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
Patch 1 #6945
Patch 1 #6945
Conversation
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
@WilliamDAssafMSFT is attempting to deploy a commit to the dbt-labs Team on Vercel. A member of the Team first needs to authorize it. |
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 did a quick scan and it looks reasonable to me.
hey @WilliamDAssafMSFT , thanks so much for opening this pr up and clarifying that info! Mike and I have reviewed and approved it - will get this merged for you so it's live ✨ ! |
What are you changing in this pull request and why?
Updates and fixes to dbt documentation based on recent Microsoft Fabric and Azure Synapse Analytics naming conventions. Bringing dbt in line with the names of Azure Synapse Analytics dedicated SQL pools and warehouses in Microsoft Fabric Data Warehouse. Note that "Synapse" has been dropped from Microsoft Fabric branding.
Checklist