|
4 | 4 | **This PR will result in the following new package version:** |
5 | 5 | <!--- Please add details around your decision for breaking vs non-breaking version upgrade. If this is a breaking change, were backwards-compatible options explored? --> |
6 | 6 |
|
7 | | -**Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:** |
| 7 | +**Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:** |
| 8 | +<!--- Copy/paste the CHANGELOG for this version below. --> |
8 | 9 |
|
9 | 10 | ## PR Checklist |
10 | 11 | ### Basic Validation |
11 | 12 | Please acknowledge that you have successfully performed the following commands locally: |
12 | | -- [ ] dbt compile |
13 | | -- [ ] dbt run –full-refresh |
14 | | -- [ ] dbt run |
15 | | -- [ ] dbt test |
16 | | -- [ ] dbt run –vars (if applicable) |
| 13 | +- [ ] dbt run –full-refresh && dbt test |
| 14 | +- [ ] dbt run (if incremental models are present) && dbt test |
17 | 15 |
|
18 | 16 | Before marking this PR as "ready for review" the following have been applied: |
19 | | -- [ ] The appropriate issue has been linked and tagged |
20 | | -- [ ] You are assigned to the corresponding issue and this PR |
| 17 | +- [ ] The appropriate issue has been linked, tagged, and properly assigned |
| 18 | +- [ ] All necessary documentation and version upgrades have been applied |
| 19 | +- [ ] docs were regenerated (unless this PR does not include any code or yml updates) |
21 | 20 | - [ ] BuildKite integration tests are passing |
| 21 | +- [ ] Detailed validation steps have been provided below |
22 | 22 |
|
23 | 23 | ### Detailed Validation |
24 | | -Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review": |
25 | | -- [ ] You have validated these changes and assure this PR will address the respective Issue/Feature. |
26 | | -- [ ] You are reasonably confident these changes will not impact any other components of this package or any dependent packages. |
27 | | -- [ ] You have provided details below around the validation steps performed to gain confidence in these changes. |
| 24 | +Please share any and all of your validation steps: |
28 | 25 | <!--- Provide the steps you took to validate your changes below. --> |
29 | 26 |
|
30 | | -### Standard Updates |
31 | | -Please acknowledge that your PR contains the following standard updates: |
32 | | -- Package versioning has been appropriately indexed in the following locations: |
33 | | - - [ ] indexed within dbt_project.yml |
34 | | - - [ ] indexed within integration_tests/dbt_project.yml |
35 | | -- [ ] CHANGELOG has individual entries for each respective change in this PR |
36 | | - <!--- If there is a parallel upstream change, remember to reference the corresponding CHANGELOG as an individual entry. --> |
37 | | -- [ ] README updates have been applied (if applicable) |
38 | | - <!--- Remember to check the following README locations for common updates. → |
39 | | - <!--- Suggested install range (needed for breaking changes) → |
40 | | - <!--- Dependency matrix is appropriately updated (if applicable) → |
41 | | - <!--- New variable documentation (if applicable) --> |
42 | | -- [ ] DECISIONLOG updates have been updated (if applicable) |
43 | | -- [ ] Appropriate yml documentation has been added (if applicable) |
44 | | - |
45 | | -### dbt Docs |
46 | | -Please acknowledge that after the above were all completed the below were applied to your branch: |
47 | | -- [ ] docs were regenerated (unless this PR does not include any code or yml updates) |
48 | | - |
49 | 27 | ### If you had to summarize this PR in an emoji, which would it be? |
50 | 28 | <!--- For a complete list of markdown compatible emojis check our this git repo (https://gist.github.com/rxaviers/7360908) --> |
51 | | -:dancer: |
| 29 | +:dancer: |
0 commit comments