Skip to content

Update sentry config#426

Merged
sislr merged 2 commits intomainfrom
sentry-config
Dec 3, 2025
Merged

Update sentry config#426
sislr merged 2 commits intomainfrom
sentry-config

Conversation

@sislr
Copy link
Member

@sislr sislr commented Dec 1, 2025

@sislr sislr requested a review from coorasse December 1, 2025 10:33
@sislr sislr self-assigned this Dec 1, 2025
Copy link
Contributor

@edmunteanu edmunteanu left a comment

Choose a reason for hiding this comment

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

Looks good! Could be potentially made more compact like this:

      event.extra = filter.filter(event.extra) if event.extra
      event.user = filter.filter(event.user) if event.user
      event.contexts = filter.filter(event.contexts) if event.contexts

      event

Copy link
Member

@coorasse coorasse left a comment

Choose a reason for hiding this comment

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

I like the compact version proposed by @edmunteanu

@sislr sislr merged commit cf71a3c into main Dec 3, 2025
1 of 2 checks passed
@sislr sislr deleted the sentry-config branch December 3, 2025 07:47
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.

3 participants