Skip to content

Conversation

@tejas7777
Copy link
Contributor

@tejas7777 tejas7777 commented Aug 14, 2025

Sentry additional data

image

@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2025

Codecov Report

❌ Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/spam/email_checker.py 87.50% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@tejas7777 tejas7777 force-pushed the extra-details-spam-email-sentry-warnings branch from c64b626 to 4007441 Compare August 15, 2025 10:01
Copy link
Contributor

@Pager07 Pager07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good work @tejas7777 , a couple of comments from me

@Pager07
Copy link
Contributor

Pager07 commented Aug 18, 2025

lgtm, testing in Beta; for some reason, not getting emails about these events in sentry; you getting thoese?

image

@Pager07 Pager07 assigned tejas7777 and unassigned Pager07 Aug 18, 2025
@tomhamiltonstubber tomhamiltonstubber merged commit f3b81f3 into master Aug 26, 2025
3 checks passed
@tomhamiltonstubber tomhamiltonstubber deleted the extra-details-spam-email-sentry-warnings branch August 26, 2025 15:54
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.

5 participants