1
85%
master: 85%

Ran 08 Jul 2026 07:44AM UTC

Files 53

Run time 2s

Badge

Embed ▾

Committed 08 Jul 2026 07:41AM UTC coverage: 84.529% (-0.006%) from 84.535%

Job # 28926099734.1

Build Type

Pull #2106

github

Committed by

paskal

Commit Message

Fix dropped notification errors and switch to errors.Join

notify/email.go accumulated multi-recipient errors with
multierror.Append(fmt.Errorf(...)) instead of
multierror.Append(result, ...), so the accumulator was overwritten each
iteration and only the last failing recipient's error survived; earlier
failures were silently dropped. The telegram notifier did it correctly.

Replace hashicorp/go-multierror with the stdlib errors.Join everywhere
it was used (notify/email.go, notify/telegram.go, rest/api/rest_private.go,
store/service/service.go, store/image/image.go and store/engine/bolt.go),
which fixes the bug and drops the direct dependency. It stays indirect
because go-pkgz/lcw/v2 still imports it. A regression test in
email_test.go now sends two failing recipients and asserts both errors
are reported.

Pull Request Pull Request #2106: Fix dropped notification errors and switch to errors.Join

Coverage Stats

6469 of 7653 relevant lines covered (84.53%)

36.76 hits per line

umputun / remark42 / 28926099734 / 1
85%
master: 85%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 28926099734.1

umputun / remark42 / 28926099734 / 1 85% master: 85%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 28926099734.1

umputun / remark42 / 28926099734 / 1
85%
master: 85%

README BADGES
x