diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2016-03-08 18:22:50 +0000 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2016-05-17 10:17:45 +0100 |
commit | 6b834f2cbcfc26fe3123b6682ed7e20618e31d1b (patch) | |
tree | 0b812d498079fdcaaf0d004a5bce716560b99be0 /app/services/merge_requests/add_todo_when_build_fails_service.rb | |
parent | 78a67fc48dab434b43a080e5b15491963656661a (diff) | |
download | gitlab-ce-6b834f2cbcfc26fe3123b6682ed7e20618e31d1b.tar.gz |
Create a todo on failing MR build
When a build fails for a commit, create a todo for the author of the
merge request that commit is the HEAD of. If the commit isn't the HEAD
commit of any MR, don't do anything. If there already is a todo for that
user and MR, don't do anything.
Current limitations:
- This isn't configurable by project.
- The author of a merge request might not be the person who pushed the
breaking commit.
Diffstat (limited to 'app/services/merge_requests/add_todo_when_build_fails_service.rb')
-rw-r--r-- | app/services/merge_requests/add_todo_when_build_fails_service.rb | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/app/services/merge_requests/add_todo_when_build_fails_service.rb b/app/services/merge_requests/add_todo_when_build_fails_service.rb new file mode 100644 index 00000000000..566049525cb --- /dev/null +++ b/app/services/merge_requests/add_todo_when_build_fails_service.rb @@ -0,0 +1,17 @@ +module MergeRequests + class AddTodoWhenBuildFailsService < MergeRequests::BaseService + # Adds a todo to the parent merge_request when a CI build fails + def execute(commit_status) + each_merge_request(commit_status) do |merge_request| + todo_service.merge_request_build_failed(merge_request) + end + end + + # Closes any pending build failed todos for the parent MRs when a build is retried + def close(commit_status) + each_merge_request(commit_status) do |merge_request| + todo_service.merge_request_build_retried(merge_request) + end + end + end +end |