From d9ab72d6080f594d0b3cae15f14b3ef2c6c638cb Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Wed, 20 Oct 2021 08:43:02 +0000 Subject: Add latest changes from gitlab-org/gitlab@14-4-stable-ee --- danger/product_intelligence/Dangerfile | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'danger/product_intelligence/Dangerfile') diff --git a/danger/product_intelligence/Dangerfile b/danger/product_intelligence/Dangerfile index ae58cf4588c..fd6ae76b4f1 100644 --- a/danger/product_intelligence/Dangerfile +++ b/danger/product_intelligence/Dangerfile @@ -5,19 +5,19 @@ CHANGED_FILES_MESSAGE = <<~MSG For the following files, a review from the [Data team and Product Intelligence team](https://gitlab.com/groups/gitlab-org/growth/product-intelligence/engineers/-/group_members?with_inherited_permissions=exclude) is recommended Please check the ~"product intelligence" [guide](https://docs.gitlab.com/ee/development/usage_ping.html). +For MR review guidelines, see the [Service Ping review guidelines](https://docs.gitlab.com/ee/development/usage_ping/review_guidelines.html) or the [Snowplow review guidelines](https://docs.gitlab.com/ee/development/snowplow/review_guidelines.html). %s MSG -# exit if not matching files +# exit if not matching files or if no product intelligence labels matching_changed_files = product_intelligence.matching_changed_files -return unless matching_changed_files.any? +labels = product_intelligence.missing_labels -warn format(CHANGED_FILES_MESSAGE, changed_files: helper.markdown_list(matching_changed_files)) +return if matching_changed_files.empty? || labels.empty? -labels = product_intelligence.missing_labels -return unless labels.any? +warn format(CHANGED_FILES_MESSAGE, changed_files: helper.markdown_list(matching_changed_files)) gitlab.api.update_merge_request(gitlab.mr_json['project_id'], gitlab.mr_json['iid'], -- cgit v1.2.1