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

ref(grouping): Add types to enhancement methods #87370

Draft
wants to merge 7 commits into
base: kmclb-enhancement-types-prework
Choose a base branch
from

Conversation

lobsterkatie
Copy link
Member

This adds type hints for all of the enhancements methods, to increase the safety of upcoming refactors.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 18, 2025
Copy link

codecov bot commented Mar 19, 2025

Codecov Report

Attention: Patch coverage is 98.59155% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/grouping/enhancer/__init__.py 92.30% 1 Missing ⚠️
Additional details and impacted files
@@                       Coverage Diff                        @@
##           kmclb-enhancement-types-prework   #87370   +/-   ##
================================================================
  Coverage                            87.73%   87.73%           
================================================================
  Files                                 9864     9864           
  Lines                               559033   559055   +22     
  Branches                             22043    22043           
================================================================
+ Hits                                490466   490492   +26     
+ Misses                               68136    68132    -4     
  Partials                               431      431           

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.

1 participant