Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: do not match archived accounts #1131

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Conversation

morremeyer
Copy link
Member

This fixes a bug where match rules targeting archived accounts were still applied, which is counterintuitive.

@morremeyer morremeyer requested a review from a team as a code owner February 12, 2025 16:51
This fixes a bug where match rules targeting archived accounts were still applied, which is counterintuitive.
@morremeyer morremeyer force-pushed the fix/match-rule-archived branch from 9642d7c to 7e5b798 Compare February 12, 2025 16:52
Copy link

codecov bot commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.39%. Comparing base (3bb89b0) to head (7e5b798).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1131   +/-   ##
=======================================
  Coverage   81.39%   81.39%           
=======================================
  Files          56       56           
  Lines        4923     4923           
=======================================
  Hits         4007     4007           
  Misses        772      772           
  Partials      144      144           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@morremeyer morremeyer merged commit 2e97209 into main Feb 12, 2025
6 checks passed
@morremeyer morremeyer deleted the fix/match-rule-archived branch February 12, 2025 16: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.

1 participant