Skip to content

Add log message when DedupeIntegration is dropping an error.#4788

Merged
antonpirker merged 1 commit intomasterfrom
antonpirker/log-dropped-event-in-dedupe-integration
Sep 11, 2025
Merged

Add log message when DedupeIntegration is dropping an error.#4788
antonpirker merged 1 commit intomasterfrom
antonpirker/log-dropped-event-in-dedupe-integration

Conversation

@antonpirker
Copy link
Copy Markdown
Contributor

@antonpirker antonpirker commented Sep 11, 2025

Make it clearer for the user what is happening.

resolves: PY-1840

@antonpirker antonpirker requested a review from a team as a code owner September 11, 2025 05:51
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.58%. Comparing base (94a92d8) to head (189db0f).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4788      +/-   ##
==========================================
+ Coverage   84.55%   84.58%   +0.03%     
==========================================
  Files         158      158              
  Lines       16479    16480       +1     
  Branches     2855     2855              
==========================================
+ Hits        13933    13939       +6     
+ Misses       1700     1695       -5     
  Partials      846      846              
Files with missing lines Coverage Δ
sentry_sdk/integrations/dedupe.py 87.87% <100.00%> (+0.37%) ⬆️

... and 2 files with indirect coverage changes

@linear
Copy link
Copy Markdown

linear Bot commented Sep 11, 2025

@antonpirker antonpirker merged commit a6a2f93 into master Sep 11, 2025
128 checks passed
@antonpirker antonpirker deleted the antonpirker/log-dropped-event-in-dedupe-integration branch September 11, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants