diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-02-19 17:38:47 +0100 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-02-19 23:24:31 +0100 |
commit | 56449cc6747224adbf3a55e9ac5d21e24dbbaa30 (patch) | |
tree | 416b5ad971cf562bbfbb42181b9fc39d16ffc1fd /app/models/commit_status.rb | |
parent | 2cc9a42ca45d14fc7fe35ea6f8bc4f9275f33144 (diff) | |
download | gitlab-ce-56449cc6747224adbf3a55e9ac5d21e24dbbaa30.tar.gz |
Fix Merge When Succeeded for multiple stages
Use around_transition to trigger build creation for next stages
Diffstat (limited to 'app/models/commit_status.rb')
-rw-r--r-- | app/models/commit_status.rb | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/app/models/commit_status.rb b/app/models/commit_status.rb index 434b3560d09..7ef50836322 100644 --- a/app/models/commit_status.rb +++ b/app/models/commit_status.rb @@ -75,16 +75,16 @@ class CommitStatus < ActiveRecord::Base transition [:pending, :running] => :canceled end - after_transition pending: :running do |build, transition| - build.update_attributes started_at: Time.now + after_transition pending: :running do |commit_status| + commit_status.update_attributes started_at: Time.now end - after_transition any => [:success, :failed, :canceled] do |build, transition| - build.update_attributes finished_at: Time.now + after_transition any => [:success, :failed, :canceled] do |commit_status| + commit_status.update_attributes finished_at: Time.now end - after_transition [:pending, :running] => :success do |build, transition| - MergeRequests::MergeWhenBuildSucceedsService.new(build.commit.project, nil).trigger(build) + after_transition [:pending, :running] => :success do |commit_status| + MergeRequests::MergeWhenBuildSucceedsService.new(commit_status.commit.project, nil).trigger(commit_status) end state :pending, value: 'pending' |