Merge branch '254237-update-danger-bot-reviews-from-telemetry-to-product-analytics-2' into 'master'

Update Danger bot reviews from Telemetry to Product Analytics

Closes #254237

See merge request gitlab-org/gitlab!42850

danger/telemetry/Dangerfile → danger/product_analytics/Dangerfile

 # frozen_string_literal: true
 
-TELEMETRY_CHANGED_FILES_MESSAGE = <<~MSG
-For the following files, a review from the [Data team and Telemetry team](https://gitlab.com/groups/gitlab-org/growth/telemetry/engineers/-/group_members?with_inherited_permissions=exclude) is recommended
-Please check the ~telemetry [guide](https://docs.gitlab.com/ee/development/telemetry/usage_ping.html) and reach out to %<telemetry_engineers_group>s group for a review.
+PRODUCT_ANALYTICS_CHANGED_FILES_MESSAGE = <<~MSG
+For the following files, a review from the [Data team and Product Analytics team](https://gitlab.com/groups/gitlab-org/growth/product_analytics/engineers/-/group_members?with_inherited_permissions=exclude) is recommended
+Please check the ~"product analytics(telemetry)" [guide](https://docs.gitlab.com/ee/development/telemetry/usage_ping.html) and reach out to %<product_analytics_engineers_group>s group for a review.
+
 
 %<changed_files>s
 
@@ -13,7 +14,7 @@ UPDATE_METRICS_DEFINITIONS_MESSAGE = <<~MSG
 
 MSG
 
-TELEMETRY_ENGINEERS_GROUP = '@gitlab-org/growth/telemetry/engineers'
+PRODUCT_ANALYTICS_ENGINEERS_GROUP = '@gitlab-org/growth/product_analytics/engineers'
 
 tracking_files = [
   'lib/gitlab/tracking.rb',
@@ -32,16 +33,16 @@ changed_files = (usage_data_changed_files + snowplow_events_changed_files)
 if changed_files.any?
 
   mention = if helper.draft_mr?
-              "`#{TELEMETRY_ENGINEERS_GROUP}`"
+              "`#{PRODUCT_ANALYTICS_ENGINEERS_GROUP}`"
             else
-              TELEMETRY_ENGINEERS_GROUP
+              PRODUCT_ANALYTICS_ENGINEERS_GROUP
             end
 
-  warn format(TELEMETRY_CHANGED_FILES_MESSAGE, changed_files: helper.markdown_list(changed_files), telemetry_engineers_group: mention)
+  warn format(PRODUCT_ANALYTICS_CHANGED_FILES_MESSAGE, changed_files: helper.markdown_list(changed_files), product_analytics_engineers_group: mention)
   warn format(UPDATE_METRICS_DEFINITIONS_MESSAGE) unless helper.changed_files(/usage_ping\.md/).any?
 
-  telemetry_labels = ['telemetry']
-  telemetry_labels << 'telemetry::review pending' unless helper.mr_has_labels?('telemetry::reviewed')
+  product_analytics_labels = ['product analytics(telemetry)']
+  product_analytics_labels << 'product analytics(telemetry)::review pending' unless helper.mr_has_labels?('product analytics(telemetry)::reviewed')
 
-  markdown(helper.prepare_labels_for_mr(telemetry_labels))
+  markdown(helper.prepare_labels_for_mr(product_analytics_labels))
 end

lib/gitlab_danger.rb

@@ -11,7 +11,7 @@ class GitlabDanger
     karma
     database
     commit_messages
-    telemetry
+    product_analytics
     utility_css
     pajamas
   ].freeze

spec/lib/gitlab_danger_spec.rb

@@ -9,7 +9,7 @@ RSpec.describe GitlabDanger do
 
   describe '.local_warning_message' do
     it 'returns an informational message with rules that can run' do
-      expect(described_class.local_warning_message).to eq('==> Only the following Danger rules can be run locally: changes_size, documentation, frozen_string, duplicate_yarn_dependencies, prettier, eslint, karma, database, commit_messages, telemetry, utility_css, pajamas')
+      expect(described_class.local_warning_message).to eq('==> Only the following Danger rules can be run locally: changes_size, documentation, frozen_string, duplicate_yarn_dependencies, prettier, eslint, karma, database, commit_messages, product_analytics, utility_css, pajamas')
     end
   end