Skip to content

Commit

Permalink
clarify property.yml function (#3669)
Browse files Browse the repository at this point in the history
users were confused with snapshot config behavior. 

clarify property.yml and remove code per [slack
thread](https://dbt-labs.slack.com/archives/C017GDLAF7D/p1688519666291969)
to help users
  • Loading branch information
mirnawong1 authored Jul 5, 2023
2 parents e0632b1 + 75622e5 commit 7601780
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions website/docs/reference/snapshot-configs.md
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,12 @@ snapshots:

<TabItem value="property-yaml">

<File name='snapshots/properties.yml'>
**Note:** Required snapshot properties _will not_ work when defined in `config` YAML blocks. We recommend that you define these in `dbt_project.yml` or a `config()` block within the snapshot `.sql` file.

**Note:** Required snapshot properties may not work when defined in `config` YAML blocks. We recommend that you define these in `dbt_project.yml` or a `config()` block within the snapshot `.sql` file.

<!--
<File name='snapshots/properties.yml'>
```yaml
version: 2
Expand All @@ -64,8 +66,8 @@ snapshots:
[check_cols](/reference/resource-configs/check_cols): [<column_name>] | all
```

</File>
-->

</TabItem>

Expand Down

0 comments on commit 7601780

Please sign in to comment.