summaryrefslogtreecommitdiff
path: root/app/services/protected_branches
diff options
context:
space:
mode:
authorYorick Peterse <yorickpeterse@gmail.com>2018-08-16 14:46:40 +0200
committerYorick Peterse <yorickpeterse@gmail.com>2018-08-16 17:29:37 +0200
commit9606dbbb033567de9831a8cdea0e56236e7d2eb2 (patch)
treea83e2103a7f3ab31836375f1adbef605f050f0f8 /app/services/protected_branches
parent96ce2da74ee36e88f20cbd7ceaff2ab49f0bb223 (diff)
downloadgitlab-ce-9606dbbb033567de9831a8cdea0e56236e7d2eb2.tar.gz
Whitelist existing destroy_all offensesblacklist-destroy-all
This whitelists all existing places where we use "destroy_all".
Diffstat (limited to 'app/services/protected_branches')
-rw-r--r--app/services/protected_branches/legacy_api_update_service.rb4
1 files changed, 2 insertions, 2 deletions
diff --git a/app/services/protected_branches/legacy_api_update_service.rb b/app/services/protected_branches/legacy_api_update_service.rb
index 1f6bbe72f85..da8bf2ce02a 100644
--- a/app/services/protected_branches/legacy_api_update_service.rb
+++ b/app/services/protected_branches/legacy_api_update_service.rb
@@ -38,11 +38,11 @@ module ProtectedBranches
def delete_redundant_access_levels
unless @developers_can_merge.nil?
- @protected_branch.merge_access_levels.destroy_all
+ @protected_branch.merge_access_levels.destroy_all # rubocop: disable DestroyAll
end
unless @developers_can_push.nil?
- @protected_branch.push_access_levels.destroy_all
+ @protected_branch.push_access_levels.destroy_all # rubocop: disable DestroyAll
end
end
end