Skip to content

ref(grouping): Expand datetime message parameterization#109706

Merged
lobsterkatie merged 2 commits intomasterfrom
kmclb-expand-datetime-parameterization
Mar 2, 2026
Merged

ref(grouping): Expand datetime message parameterization#109706
lobsterkatie merged 2 commits intomasterfrom
kmclb-expand-datetime-parameterization

Conversation

@lobsterkatie
Copy link
Member

This refactors our datetime parameterization regexes to combine one set of the "old, potentially-removable" patterns (which has been spotted in the wild, so no, we can't remove it) with the current dash-optional pattern, and then expands the resulting pattern set so the colons in the time are optional as well. It then splits the set in two for greater readability. The resulting pattern set now handles the following:

  • Date with or without dashes and time with or without colons

  • Date and time separated either by a space or by T

  • Decimal seconds, integer seconds, or no seconds

  • Timezone information given as offset, as Z (UTC), or not given at all

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 2, 2026
@lobsterkatie lobsterkatie marked this pull request as ready for review March 2, 2026 21:06
@lobsterkatie lobsterkatie requested a review from a team as a code owner March 2, 2026 21:06
@lobsterkatie lobsterkatie merged commit ebbca5d into master Mar 2, 2026
76 checks passed
@lobsterkatie lobsterkatie deleted the kmclb-expand-datetime-parameterization branch March 2, 2026 21:25
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