Skip to content

Conversation

szokeasaurusrex
Copy link
Member

Previous name said that this filter was "hub-based," when the logic in reality is not related to hubs. So, we should rename the filter to something more sensible.

Previous name said that this filter was "hub-based," when the logic in reality is not related to hubs. So, we should rename the filter to something more sensible.
@szokeasaurusrex szokeasaurusrex self-assigned this Jul 9, 2024
Copy link
Member

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

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

💯

Copy link

codecov bot commented Jul 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.38%. Comparing base (9d97d93) to head (4965aa8).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3260      +/-   ##
==========================================
+ Coverage   79.37%   79.38%   +0.01%     
==========================================
  Files         132      132              
  Lines       14263    14263              
  Branches     2992     2992              
==========================================
+ Hits        11321    11323       +2     
  Misses       2094     2094              
+ Partials      848      846       -2     
Files Coverage Δ
sentry_sdk/debug.py 96.29% <100.00%> (ø)

... and 2 files with indirect coverage changes

@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) July 10, 2024 14:09
@szokeasaurusrex szokeasaurusrex disabled auto-merge July 10, 2024 14:12
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) July 11, 2024 07:52
@szokeasaurusrex szokeasaurusrex merged commit c359c82 into master Jul 11, 2024
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/debug-filter branch July 11, 2024 08:03
arjennienhuis pushed a commit to arjennienhuis/sentry-python that referenced this pull request Sep 30, 2024
Previous name said that this filter was "hub-based," when the logic in reality is not related to hubs. So, we should rename the filter to something more sensible.
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