Merge branch 'main' into config/triage-actions

.github/ISSUE_TEMPLATE/internal-epic.yml

@@ -29,13 +29,36 @@ body:
     attributes:
       label: Objectives
       description: |
-        What are the goals we are trying to achieve? Provide use cases if available.
-      placeholder: |
-        - [ ] Add option to ...
-        - [ ] Allow adapter maintainers to configure ...
+        What are the high level goals we are trying to achieve? Provide use cases if available.
+        
+        Example:
+        - [ ] Allow adapter maintainers to support custom materializations
+        - [ ] Reduce maintenance burden for incremental users by offering materialized views
+      value: |
+        ```[tasklist]
+        - [ ] Objective
+        ```
     validations:
       required: true
 
+  - type: textarea
+    attributes:
+      label: Implementation tasks
+      description: |
+        Provide a list of GH issues that will build out this functionality.
+        This may start empty, or as a checklist of items.
+        However, it should eventually become a list of Feature Implementation tickets.
+        
+        Example:
+        - [ ] Create new macro to select warehouse
+        - [ ] https://github.com/dbt-labs/dbt-adapters/issues/42
+      value: |
+        ```[tasklist]
+        - [ ] Task
+        ```
+    validations:
+      required: false
+
   - type: textarea
     attributes:
       label: Consequences
@@ -48,19 +71,3 @@ body:
         - This allows internal analytics to do more robust analysis of infrastructure usage (Add the `Impact:[Analytics]` label.)
     validations:
       required: true
-
-  - type: textarea
-    attributes:
-      label: Implementation tasks
-      description: |
-        Provide a list of GH issues that will build out this functionality. This may start empty.
-      placeholder: |
-        - [ ] https://github.com/dbt-labs/dbt-adapters/issues/123
-        - [ ] https://github.com/dbt-labs/dbt-postgres/issues/456
-      value: |
-        ```[tasklist]
-        ### Implementation tasks
-        - [ ]
-        ```
-    validations:
-      required: false

.github/ISSUE_TEMPLATE/internal-feature-implementation.yml

@@ -41,17 +41,31 @@ body:
       label: Acceptance criteria
       description: |
         What is the definition of done for this feature? Include any relevant edge cases and/or test cases.
+        
+        Example:
+        - [ ] If there are no config changes, don't alter the materialized view
+        - [ ] If the materialized view is scheduled to refresh, a manual refresh should not be issued
+      value: |
+        ```[tasklist]
+        - [ ] Criterion
+        ```
     validations:
       required: true
 
   - type: textarea
     attributes:
       label: Suggested tests
       description: |
-        Provide scenarios to test. Include both positive and negative tests if possible. Link to existing similar tests if appropriate.
-      placeholder: |
-         1. Test with no `materialized` field in the model config. Expect pass.
-         2. Test with a `materialized` field in the model config that is not valid. Expect ConfigError.
+        Provide scenarios to test. Include both positive and negative tests if possible.
+        Link to existing similar tests if appropriate.
+        
+        Example:
+         - [ ] Test with no `materialized` field in the model config. Expect pass.
+         - [ ] Test with a `materialized` field in the model config that is not valid. Expect ConfigError.
+      value: |
+        ```[tasklist]
+         - [ ] Test
+        ```
     validations:
       required: true
 
@@ -62,6 +76,7 @@ body:
         Will this impact dbt Labs' ability, or a partner's ability, to make a related change? Call that out for discussion.
         Review `Impact:<team>` labels to ensure they capture these consequences.
       placeholder: |
-        Example: This change impacts `dbt-databricks` because we updated a macro in `dbt-spark`. (Add the `Impact:[Databricks]` label.)
+        Example:
+        - This change impacts `dbt-databricks` because we updated a macro in `dbt-spark`. (Add the `Impact:[Databricks]` label.)
     validations:
       required: true