summaryrefslogtreecommitdiff
path: root/app/services/delete_merged_branches_service.rb
Commit message (Collapse)AuthorAgeFilesLines
* Replace deprecated uniq on a Relation with distinctJasper Maes2018-10-261-2/+2
|
* Disable existing offenses for the CodeReuse copsYorick Peterse2018-09-111-0/+2
| | | | | This whitelists all existing offenses for the various CodeReuse cops, of which most are triggered by the CodeReuse/ActiveRecord cop.
* Enable frozen string in apps/uploaders/*.rbgfyoung2018-07-161-0/+2
| | | | Partially addresses #47424.
* Remove allow_n_plus_1 block from DeleteMergedBranchesServicefix/remove-allow-n-plus-1-from-delete-merged-branches-serviceAhmad Sherif2018-02-121-11/+7
| | | | | Fixes #37438 Fixes gitaly#999
* When deleting merged branches, ignore protected tagstc-delete-merged-protected-tags-fixToon Claes2017-11-071-1/+1
| | | | | | | | | | | | In gitlab-org/gitlab-ce!13251 wildcard Protected Branches were handled properly when deleting all merged branches. But this fix wasn't that good. It also checked branch names against Protected Tags. That's not correct. This change will **only** check if there is a Protected Branch matching the merged branch, and ignores Protected Tags. Closes gitlab-org/gitlab-ce#39732.
* Detect n+1 issues involving GitalyAndrew Newdigate2017-09-191-8/+11
|
* Make Delete Merged Branches handle wildcard protected branches correctlyToon Claes2017-08-021-1/+1
| | | | | | | The "Delete Merged Branches" button should filter out protected branches matching the wildcard patterns. Closes gitlab-org/gitlab-ce#35592.
* DeleteMergedBranchesService should not delete protected branchesToon Claes2017-07-041-0/+2
| | | | | When deleting all the branches that are merged, the protected branches should not be deleted.
* Don't delete a branch involved in an open merge request in "Delete all ↵sh-issue-29247-fixStan Hu2017-04-191-0/+11
| | | | | | | | | merged branches" service Customers were surprised by the previous behavior, which destroyed branches even though an open merge request existed for it. Closes #29427
* Remove unnecessary require_relative calls from service classesSemyon Pupkov2016-11-221-2/+0
| | | | | Rails by default use autoload for all dirs from app folder. require_relative not needed. See ActiveSupport::Dependencies.autoload_paths
* Add button to delete all merged branchesToon Claes2016-11-091-0/+18
It adds a button to the branches page that the user can use to delete all the branches that are already merged. This can be used to clean up all the branches that were forgotten to delete while merging MRs. Fixes #21076.
View Lorry Controller Status