fix: incident merging

[Matvey-Kuk]

close #3370

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
keep	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 19, 2025 7:09pm

[talboren]

What's the outcome of this PR, I don't understand what will happen with those skipped incidents?

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 31.12%. Comparing base (d667aaf) to head (a2b0d49).
Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
keep/api/core/db.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #4058       +/-   ##
===========================================
- Coverage   47.03%   31.12%   -15.92%     
===========================================
  Files         165       92       -73     
  Lines       16210    10060     -6150     
===========================================
- Hits         7625     3131     -4494     
+ Misses       8585     6929     -1656     

☔ View full report in Codecov by Sentry.
📢 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Matvey-Kuk]

@talboren I removed skipped incidents, it will fail only on the case of exception. This PR will allow merging incidents without alerts & will fix the case when UI says incidents are merged but they are not.

[talboren]

LGTM