Skip to content

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

Merged
antonpirker merged 1 commit into
masterfrom
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 into
masterfrom
antonpirker/log-dropped-event-in-dedupe-integration

Conversation

@antonpirker

@antonpirker antonpirker commented Sep 11, 2025

Copy link
Copy Markdown
Contributor

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

codecov Bot commented Sep 11, 2025

Copy link
Copy Markdown

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

linear Bot commented Sep 11, 2025

Copy link
Copy Markdown

@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