Commit Graph

11 Commits

Author SHA1 Message Date
Lin Jen-Shin 2e067480a9 Rename for a more consistent pipeline worker name, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342/diffs#note_17039533
2016-10-17 18:12:07 +08:00
Lin Jen-Shin 8dd580f402 It's already async, from Project#execute_services, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342/diffs#note_16132555
2016-10-17 07:25:21 +00:00
Lin Jen-Shin eeeb96c9d0 Change service to be a worker, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342#note_16118195
2016-10-17 07:25:20 +00:00
Lin Jen-Shin bac500fdeb It no longer needs a block, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342#note_15434727
2016-10-17 07:25:20 +00:00
Lin Jen-Shin 9c2d40649e Remove pusher support because it's covered by notifications 2016-10-17 07:25:20 +00:00
Lin Jen-Shin f1e5ca3a31 Use any? instead of count > 0, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019#note_16832243
2016-10-13 01:12:00 +08:00
Lin Jen-Shin 9d92d41ff9 Omit else, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019/diffs#note_15336254
2016-09-15 00:00:47 +08:00
Lin Jen-Shin 844d4e24af Use keyword arg, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019/diffs#note_15336212
2016-09-15 00:00:02 +08:00
Lin Jen-Shin 4ad63c29bb Fixed the names and add missing check 2016-09-13 23:38:46 +08:00
Lin Jen-Shin 4add6ca6ec Try to integrate the email into notification system 2016-09-13 19:09:53 +08:00
Lin Jen-Shin 34cbc51e23 Add a new pipeline email service. TODO: email templates and tests 2016-08-26 16:56:55 +08:00