Skip to content

ref(grouping): Move message normalization to utils#107364

Merged
lobsterkatie merged 6 commits intomasterfrom
kmclb-move-message-normalization-to-utils
Jan 30, 2026
Merged

ref(grouping): Move message normalization to utils#107364
lobsterkatie merged 6 commits intomasterfrom
kmclb-move-message-normalization-to-utils

Conversation

@lobsterkatie
Copy link
Member

@lobsterkatie lobsterkatie commented Jan 30, 2026

This moves the normalize_message_for_grouping helper (and the REGEX_PATTERN_KEYS constant it uses) from the message strategy module to the utils module, since it is soon going to be used in more places besides just the message strategy. A few cosmetic changes (tweaking the docstring, initializing the parameterizer earlier, adding a TODO, etc.) have also been made.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 30, 2026
@lobsterkatie lobsterkatie force-pushed the kmclb-move-message-normalization-to-utils branch from c14ae12 to 284e895 Compare January 30, 2026 21:07
@lobsterkatie lobsterkatie marked this pull request as ready for review January 30, 2026 21:59
@lobsterkatie lobsterkatie requested a review from a team as a code owner January 30, 2026 21:59
@lobsterkatie lobsterkatie merged commit ae4766e into master Jan 30, 2026
75 checks passed
@lobsterkatie lobsterkatie deleted the kmclb-move-message-normalization-to-utils branch January 30, 2026 22:00
priscilawebdev pushed a commit that referenced this pull request Feb 2, 2026
This moves the `normalize_message_for_grouping` helper (and the `REGEX_PATTERN_KEYS` constant it uses) from the message strategy module to the utils module, since it is soon going to be used in more places besides just the message strategy. A few cosmetic changes (tweaking the docstring, initializing the parameterizer earlier, adding a TODO, etc.) have also been made.
@github-actions github-actions bot locked and limited conversation to collaborators Feb 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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