Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce
This commit is contained in:
commit
c3c9703437
3 changed files with 13 additions and 9 deletions
|
@ -32,6 +32,7 @@ v 7.10.0 (unreleased)
|
||||||
- Add inifinite scroll to user page activity
|
- Add inifinite scroll to user page activity
|
||||||
- Don't show commit comment button when user is not signed in.
|
- Don't show commit comment button when user is not signed in.
|
||||||
- Don't include system notes in issue/MR comment count.
|
- Don't include system notes in issue/MR comment count.
|
||||||
|
- Don't mark merge request as updated when merge status relative to target branch changes.
|
||||||
|
|
||||||
v 7.9.0
|
v 7.9.0
|
||||||
- Send EmailsOnPush email when branch or tag is created or deleted.
|
- Send EmailsOnPush email when branch or tag is created or deleted.
|
||||||
|
|
|
@ -105,6 +105,15 @@ class MergeRequest < ActiveRecord::Base
|
||||||
state :unchecked
|
state :unchecked
|
||||||
state :can_be_merged
|
state :can_be_merged
|
||||||
state :cannot_be_merged
|
state :cannot_be_merged
|
||||||
|
|
||||||
|
around_transition do |merge_request, transition, block|
|
||||||
|
merge_request.record_timestamps = false
|
||||||
|
begin
|
||||||
|
block.call
|
||||||
|
ensure
|
||||||
|
merge_request.record_timestamps = true
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
validates :source_project, presence: true, unless: :allow_broken
|
validates :source_project, presence: true, unless: :allow_broken
|
||||||
|
|
|
@ -53,7 +53,7 @@ module MergeRequests
|
||||||
|
|
||||||
if merge_request.source_branch == @branch_name || force_push?
|
if merge_request.source_branch == @branch_name || force_push?
|
||||||
merge_request.reload_code
|
merge_request.reload_code
|
||||||
update_merge_request(merge_request)
|
merge_request.mark_as_unchecked
|
||||||
else
|
else
|
||||||
mr_commit_ids = merge_request.commits.map(&:id)
|
mr_commit_ids = merge_request.commits.map(&:id)
|
||||||
push_commit_ids = @commits.map(&:id)
|
push_commit_ids = @commits.map(&:id)
|
||||||
|
@ -61,20 +61,14 @@ module MergeRequests
|
||||||
|
|
||||||
if matches.any?
|
if matches.any?
|
||||||
merge_request.reload_code
|
merge_request.reload_code
|
||||||
update_merge_request(merge_request)
|
merge_request.mark_as_unchecked
|
||||||
else
|
else
|
||||||
update_merge_request(merge_request)
|
merge_request.mark_as_unchecked
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def update_merge_request(merge_request)
|
|
||||||
MergeRequests::UpdateService.new(
|
|
||||||
merge_request.target_project,
|
|
||||||
@current_user, merge_status: 'unchecked').execute(merge_request)
|
|
||||||
end
|
|
||||||
|
|
||||||
# Add comment about pushing new commits to merge requests
|
# Add comment about pushing new commits to merge requests
|
||||||
def comment_mr_with_commits
|
def comment_mr_with_commits
|
||||||
merge_requests = @project.origin_merge_requests.opened.where(source_branch: @branch_name).to_a
|
merge_requests = @project.origin_merge_requests.opened.where(source_branch: @branch_name).to_a
|
||||||
|
|
Loading…
Reference in a new issue