Commit 702ecb2a authored by Kati Paizee's avatar Kati Paizee

Merge branch 'am-update-pi-review-docs' into 'master'

Update Product Intelligence review guide

See merge request gitlab-org/gitlab!67005
parents 7c85f127 a1797d55
...@@ -34,9 +34,14 @@ any of the following Service Ping files: ...@@ -34,9 +34,14 @@ any of the following Service Ping files:
#### The merge request **author** should #### The merge request **author** should
- Decide whether a Product Intelligence review is needed. - Decide whether a Product Intelligence review is needed. You can skip the Product Intelligence
review and remove the labels if the changes are not related to the Product Intelligence domain and
are regular backend changes.
- If a Product Intelligence review is needed, add the labels - If a Product Intelligence review is needed, add the labels
`~product intelligence` and `~product intelligence::review pending`. `~product intelligence` and `~product intelligence::review pending`.
- For merge requests authored by Product Intelligence team members:
- Assign both the `~backend` and `~product intelligence` reviews to another Product Intelligence team member.
- Assign the maintainer review to someone outside of the Product Intelligence group.
- Assign an - Assign an
[engineer](https://gitlab.com/groups/gitlab-org/growth/product-intelligence/engineers/-/group_members?with_inherited_permissions=exclude) from the Product Intelligence team for a review. [engineer](https://gitlab.com/groups/gitlab-org/growth/product-intelligence/engineers/-/group_members?with_inherited_permissions=exclude) from the Product Intelligence team for a review.
- Set the correct attributes in the metric's YAML definition: - Set the correct attributes in the metric's YAML definition:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment