Skip to content

Commit bd11e34

Browse files
committed
address PR comment
1 parent 56bfd0e commit bd11e34

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

Diff for: warehouse/scripts/templates/ci_report.md

+8-9
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
11
Warehouse report 📦
22

3+
{% if new_models or changed_incremental_models %}
34
### Checks/potential follow-ups
45

5-
{% if new_models %}
6-
7-
* All new models have a surrogate primary key that is tested to be not-null and unique
8-
9-
{% endif %}
10-
{% if changed_incremental_models %}
11-
12-
* Full refresh as a follow-up action item
13-
6+
Checks indicate the following action items may be necessary.
7+
{% if new_models -%}
8+
- [ ] For new models, do they all have a surrogate primary key that is tested to be not-null and unique?
9+
{%- endif %}
10+
{% if changed_incremental_models -%}
11+
- [ ] For changed incremental models, does the PR description identify whether a full refresh is needed for these tables?
12+
{%- endif %}
1413
{% endif %}
1514

1615

0 commit comments

Comments
 (0)