diff options
| author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-05-20 17:34:42 +0300 |
|---|---|---|
| committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-05-20 17:34:42 +0300 |
| commit | 9f46488805e86b1bc341ea1620b866016c2ce5ed (patch) | |
| tree | f9748c7e287041e37d6da49e0a29c9511dc34768 /danger/specs | |
| parent | dfc92d081ea0332d69c8aca2f0e745cb48ae5e6d (diff) | |
Add latest changes from gitlab-org/gitlab@13-0-stable-ee
Diffstat (limited to 'danger/specs')
| -rw-r--r-- | danger/specs/Dangerfile | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/danger/specs/Dangerfile b/danger/specs/Dangerfile index 50887c44b3e..7803fad8472 100644 --- a/danger/specs/Dangerfile +++ b/danger/specs/Dangerfile @@ -7,14 +7,10 @@ That's OK as long as you're refactoring existing code, but please consider adding any of the %<labels>s labels. MSG -def presented_no_changelog_labels - NO_SPECS_LABELS.map { |label| "~#{label}" }.join(', ') -end - has_app_changes = !helper.all_changed_files.grep(%r{\A(ee/)?(app|lib|db/(geo/)?(post_)?migrate)/}).empty? has_spec_changes = !helper.all_changed_files.grep(%r{\A(ee/)?spec/}).empty? new_specs_needed = (gitlab.mr_labels & NO_SPECS_LABELS).empty? if has_app_changes && !has_spec_changes && new_specs_needed - warn format(NO_NEW_SPEC_MESSAGE, labels: presented_no_changelog_labels), sticky: false + warn format(NO_NEW_SPEC_MESSAGE, labels: helper.labels_list(NO_SPECS_LABELS)), sticky: false end |