Skip to content

Added notification batching with per-messenger rate limits - #255

Merged
nt0xa merged 6 commits into
mainfrom
feature/notification-batching
Oct 7, 2026
Merged

nt0xa merged 6 commits into
mainfrom
feature/notification-batching

Moved notification batcher settings to run.go

80469a8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 7, 2026 in 1s

44.06% (+0.41%) compared to 40326d9

View this Pull Request on Codecov

44.06% (+0.41%) compared to 40326d9

Details

Codecov Report

❌ Patch coverage is 54.28571% with 64 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.06%. Comparing base (40326d9) to head (80469a8).

Files with missing lines Patch % Lines
internal/cmd/server/events.go 50.00% 14 Missing and 1 partial ⚠️
internal/modules/slack/notifier.go 0.00% 15 Missing ⚠️
internal/modules/lark/notifier.go 0.00% 13 Missing ⚠️
internal/modules/telegram/notifier.go 0.00% 13 Missing ⚠️
internal/cmd/server/run.go 0.00% 6 Missing ⚠️
internal/templates/templates.go 77.77% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
+ Coverage   43.65%   44.06%   +0.41%     
==========================================
  Files         216      216              
  Lines        8176     8305     +129     
==========================================
+ Hits         3569     3660      +91     
- Misses       4326     4360      +34     
- Partials      281      285       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.