diff options
| author | Stan Hu <stanhu@gmail.com> | 2016-01-08 17:31:35 +0000 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2016-01-08 17:31:35 +0000 |
| commit | 7403df6ca7bd31cd002d6fcf3bf4aa02dd4478eb (patch) | |
| tree | 199fcaf1278a6e1b87f57cbd7cd43e172b382561 /lib | |
| parent | 8429f6f409eb8911d21095bc0fff3db2e53efd22 (diff) | |
| parent | 69209612e1793fcebcdb784074056d7a02b0f6f7 (diff) | |
| download | gitlab-ce-7403df6ca7bd31cd002d6fcf3bf4aa02dd4478eb.tar.gz | |
Merge branch 'suppress-allow-failure-builds' into 'master'
Suppress e-mails on failed builds if allow_failure is set
Every time I push to GitLab, I get > 2 emails saying a spec failed when I don't care about the benchmarks and others that have `allow_failure` set to `true`.
@ayufan mentioned creating a summary e-mail to prevent getting one e-mail per build, but the latter might actually be desirable. For example, I do want to know if Rubocop errors fail right away.
See merge request !2178
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/gitlab/build_data_builder.rb | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/gitlab/build_data_builder.rb b/lib/gitlab/build_data_builder.rb index 86bfa0a4378..34e949130da 100644 --- a/lib/gitlab/build_data_builder.rb +++ b/lib/gitlab/build_data_builder.rb @@ -23,6 +23,7 @@ module Gitlab build_started_at: build.started_at, build_finished_at: build.finished_at, build_duration: build.duration, + build_allow_failure: build.allow_failure, # TODO: do we still need it? project_id: project.id, |