-
Notifications
You must be signed in to change notification settings - Fork 926
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
Added a caution callout for updated_at #6117
base: current
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: Matt Shaver <[email protected]>
@@ -27,6 +27,12 @@ snapshots: | |||
|
|||
</File> | |||
|
|||
:::caution | |||
|
|||
You might get a warning if the data type of the `updated_at` column does not match the adapter-configured default. |
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.
@dbeatty10 Hi! This is for this warning - what do you think about this wording?
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.
Is there any scenario in which they wouldn't get a warning about the mismatch?
If not, then I'd suggest updating "might" to be "will" instead.
Other than that, this call-out looks good! 👍
I've created this pull request following the Git issue: https://github.com/orgs/dbt-labs/projects/14/views/35?pane=issue&itemId=78645770
I've added added a caution callout / warning if data-type for updated_at doesn't match adapter configured default
Resolves #6012
What are you changing in this pull request and why?
Checklist