summaryrefslogtreecommitdiff
path: root/tooling
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2021-05-25 19:28:50 +0000
committerGitLab Bot <gitlab-bot@gitlab.com>2021-05-25 19:28:50 +0000
commitfdcb3d47df0526a9dc437c847683df8aae1e5d58 (patch)
tree3f3d01dd51485a538c987b8e9063c3ca531c944c /tooling
parent01ca8e9ec4b98f078a8633ec910d0bbc78446fed (diff)
downloadgitlab-ce-fdcb3d47df0526a9dc437c847683df8aae1e5d58.tar.gz
Add latest changes from gitlab-org/gitlab@13-12-stable-ee
Diffstat (limited to 'tooling')
-rw-r--r--tooling/danger/changelog.rb6
1 files changed, 0 insertions, 6 deletions
diff --git a/tooling/danger/changelog.rb b/tooling/danger/changelog.rb
index 44b62dcc69e..e11799b268d 100644
--- a/tooling/danger/changelog.rb
+++ b/tooling/danger/changelog.rb
@@ -65,7 +65,6 @@ module Tooling
SUGGEST_COMMENT
REQUIRED_CHANGELOG_REASONS = {
- db_changes: 'introduces a database migration',
feature_flag_removed: 'removes a feature flag'
}.freeze
REQUIRED_CHANGELOG_MESSAGE = {
@@ -235,16 +234,11 @@ module Tooling
check_result.warning("This MR has a Changelog file in `ee/`, but no code changes in `ee/`. Consider moving the Changelog file outside `ee/`.")
end
- if ee_changes.any? && ee_changelog? && required_reasons.include?(:db_changes)
- check_result.warning("This MR has a Changelog file inside `ee/`, but there are database changes which [requires](https://docs.gitlab.com/ee/development/changelog.html#what-warrants-a-changelog-entry) the Changelog placement to be outside of `ee/`. Consider moving the Changelog file outside `ee/`.")
- end
-
check_result
end
def required_reasons
[].tap do |reasons|
- reasons << :db_changes if project_helper.changes.added.has_category?(:migration)
reasons << :feature_flag_removed if project_helper.changes.deleted.has_category?(:feature_flag)
end
end