Commit ed9dd7ca authored by Rémy Coutable's avatar Rémy Coutable

Merge branch...

Merge branch '210386-update-danger-bot-to-mention-telemetry-group-instead-of-requiring-review-approval' into 'master'

Remove required telemetry approval in Danger bot

Closes #210386

See merge request gitlab-org/gitlab!27195
parents 149154f6 60578220
# frozen_string_literal: true # frozen_string_literal: true
TELEMETRY_CHANGED_FILES_MESSAGE = <<~MSG TELEMETRY_CHANGED_FILES_MESSAGE = <<~MSG
This merge request adds or changes files that require a This merge request adds or changes files for which a
review from the Data team and Telemetry team @gitlab-org/growth/telemetry. review from the Data team and Telemetry team is recommended.
The specific group is mentioned in order to send a notification to team members. @gitlab-org/growth/telemetry group is mentioned in order to notify team members.
MSG MSG
usage_data_changed_files = git.modified_files.grep(%r{usage_data}) usage_data_changed_files = git.modified_files.grep(%r{usage_data})
...@@ -12,7 +12,7 @@ if usage_data_changed_files.any? ...@@ -12,7 +12,7 @@ if usage_data_changed_files.any?
warn format(TELEMETRY_CHANGED_FILES_MESSAGE) warn format(TELEMETRY_CHANGED_FILES_MESSAGE)
USAGE_DATA_FILES_MESSAGE = <<~MSG USAGE_DATA_FILES_MESSAGE = <<~MSG
The following files require a review from the [Data team and Telemetry team](https://gitlab.com/groups/gitlab-org/growth/telemetry/-/group_members?with_inherited_permissions=exclude): For the following files, a review from the [Data team and Telemetry team](https://gitlab.com/groups/gitlab-org/growth/telemetry/-/group_members?with_inherited_permissions=exclude) is recommended:
MSG MSG
markdown(USAGE_DATA_FILES_MESSAGE + helper.markdown_list(usage_data_changed_files)) markdown(USAGE_DATA_FILES_MESSAGE + helper.markdown_list(usage_data_changed_files))
......
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