Skip to content

Conversation

JerrySentry
Copy link
Contributor

@JerrySentry JerrySentry commented Sep 16, 2025

Make the input to query to get issues by exception type to be case insensitive and ignore non alphanumeric characters.

For example when querying "ReactErrorBoundaryError" previously would not match the issues with "React Error Boundary Error" as the type in the DB

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 16, 2025
@JerrySentry JerrySentry marked this pull request as ready for review September 16, 2025 19:48
@JerrySentry JerrySentry requested a review from a team as a code owner September 16, 2025 19:48
cursor[bot]

This comment was marked as outdated.

Copy link

codecov bot commented Sep 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #99617      +/-   ##
==========================================
+ Coverage   79.34%   81.25%   +1.91%     
==========================================
  Files        8595     8600       +5     
  Lines      380806   380955     +149     
  Branches    24129    24129              
==========================================
+ Hits       302151   309555    +7404     
+ Misses      78293    71038    -7255     
  Partials      362      362              

Copy link
Contributor

@kddubey kddubey left a comment

Choose a reason for hiding this comment

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

ty for extensive tests. they are written a little repetitively tho. can separate out a helper, or maybe use pytest.mark.parametrize

@JerrySentry JerrySentry merged commit e23a773 into master Sep 17, 2025
64 checks passed
@JerrySentry JerrySentry deleted the jerry/bug-pred-tool branch September 17, 2025 14:44
Copy link

sentry-io bot commented Sep 21, 2025

Issues attributed to commits in this pull request

This pull request was merged and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants