diff --git a/pr_templates/dbt_modeling_pr.md b/pr_templates/dbt_modeling_pr.md index c153f54..3ef1972 100644 --- a/pr_templates/dbt_modeling_pr.md +++ b/pr_templates/dbt_modeling_pr.md @@ -2,15 +2,38 @@ _Summarize the goal of your work and what motivated it._ ## Key changes -- _Describe major updates._ -- _If the PR creates or renames models or adds references between models, paste a screenshot of the relevant parts of the lineage graph._ +- _Describe major updates. If multiple models or layers of models are added or updated, break up those layers/models into subheaders within this section._ +- _If applicable, include a screenshot of the affected parts of the lineage graph._ + +## Validation +- _Describe the validation process underwent to validate the changes_ +- _If dbt models were added or updated in this PR, include the queries below used to validate those changes_ + +
+ SQL validation queries + + Paste your SQL queries here + ``` + select * from table + ``` + +
+ +## Areas for reviewers to focus on +- _Highlight any outstanding questions related to your PR or any parts you'd like reviewers to take a closer look at_ + +## Future considerations +- _Call out any TODOs, known limitations, technical debt, or improvements you'd like to make at a later date_ + +## Deployment Notes +- _Specify any deployment notes or downstream impacts reviewers should be aware of_ ## Other notes -- _List open questions and other things your reviewer should know._ +- _List any other notes that don't fit into any of the above sections_ ## Related links - _List links to related ticket(s)._ -- _(optional) Link to other files and resources, such as specific related reports in a BI tool._ +- _(optional) Link to other files and resources, such as specific related reports in a BI tool or documentation._ ## Checklist - [ ] All models, whether directly changed by this PR or not, run successfully.