Commit 2fa8faa7 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Merge branch...

Merge branch 'do-not-post-database-review-required-msg-when-database-reviewed-label-is-set' into 'master'

Danger: don't require a review when 'database::approved' is set

See merge request gitlab-org/gitlab!26576
parents f3ed181b 1d595e9a
......@@ -29,6 +29,8 @@ DB_FILES_MESSAGE = <<~MSG
The following files require a review from the Database team:
MSG
DATABASE_APPROVED_LABEL = 'database::approved'
non_geo_db_schema_updated = !git.modified_files.grep(%r{\Adb/schema\.rb}).empty?
geo_db_schema_updated = !git.modified_files.grep(%r{\Aee/db/geo/schema\.rb}).empty?
......@@ -46,6 +48,7 @@ if geo_migration_created && !geo_db_schema_updated
end
return unless gitlab_danger.ci?
return if gitlab.mr_labels.include?(DATABASE_APPROVED_LABEL)
db_paths_to_review = helper.changes_by_category[:database]
......
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