diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-05-25 19:28:50 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-05-25 19:28:50 +0000 |
commit | fdcb3d47df0526a9dc437c847683df8aae1e5d58 (patch) | |
tree | 3f3d01dd51485a538c987b8e9063c3ca531c944c /tooling | |
parent | 01ca8e9ec4b98f078a8633ec910d0bbc78446fed (diff) | |
download | gitlab-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.rb | 6 |
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 |