Commit 73667ddb authored by Alina Mihaila's avatar Alina Mihaila Committed by alinamihaila

Revert "Update Danger from Telemetry to Product analytics"

This reverts commit 22392f27a75a7944ceb92a04cec34dba373124dc
parent 0e6933dc
# frozen_string_literal: true # frozen_string_literal: true
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/telemetry/engineers/-/group_members?with_inherited_permissions=exclude) is recommended TELEMETRY_CHANGED_FILES_MESSAGE = <<~MSG
Please check the ~product_analytics [guide](https://docs.gitlab.com/ee/development/telemetry/usage_ping.html) and reach out to @gitlab-org/growth/telemetry/engineers group for a review. 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 @gitlab-org/growth/telemetry/engineers group for a review.
%<changed_files>s %<changed_files>s
...@@ -27,11 +28,11 @@ snowplow_events_changed_files = git.modified_files & tracking_files ...@@ -27,11 +28,11 @@ snowplow_events_changed_files = git.modified_files & tracking_files
changed_files = (usage_data_changed_files + snowplow_events_changed_files) changed_files = (usage_data_changed_files + snowplow_events_changed_files)
if changed_files.any? if changed_files.any?
warn format(PRODUCT_ANALYTICS_CHANGED_FILES_MESSAGE, changed_files: helper.markdown_list(changed_files)) warn format(TELEMETRY_CHANGED_FILES_MESSAGE, changed_files: helper.markdown_list(changed_files))
warn format(UPDATE_METRICS_DEFINITIONS_MESSAGE) unless helper.changed_files(/usage_ping\.md/).any? warn format(UPDATE_METRICS_DEFINITIONS_MESSAGE) unless helper.changed_files(/usage_ping\.md/).any?
product_analytics_labels = ['product_analytics'] telemetry_labels = ['telemetry']
product_analytics_labels << 'product_analytics::review pending' unless helper.mr_has_labels?('product_analytics::reviewed') telemetry_labels << 'telemetry::review pending' unless helper.mr_has_labels?('telemetry::reviewed')
markdown(helper.prepare_labels_for_mr(product_analytics_labels)) markdown(helper.prepare_labels_for_mr(telemetry_labels))
end end
...@@ -11,7 +11,7 @@ class GitlabDanger ...@@ -11,7 +11,7 @@ class GitlabDanger
karma karma
database database
commit_messages commit_messages
product_analytics telemetry
utility_css utility_css
pajamas pajamas
].freeze ].freeze
......
...@@ -9,7 +9,7 @@ RSpec.describe GitlabDanger do ...@@ -9,7 +9,7 @@ RSpec.describe GitlabDanger do
describe '.local_warning_message' do describe '.local_warning_message' do
it 'returns an informational message with rules that can run' 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, product_analytics, 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, telemetry, utility_css, pajamas')
end end
end end
......
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