Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve regexes #7900

Merged
merged 2 commits into from
Nov 5, 2024
Merged

Improve regexes #7900

merged 2 commits into from
Nov 5, 2024

Conversation

liviuconcioiu
Copy link
Collaborator

@liviuconcioiu liviuconcioiu commented Nov 2, 2024

Far from complete, but I've replaced those:

[\d\.]       => [\d.]
[\.\d]       => [.\d]
[a-z0-9_\-]  => [a-z0-9_-]
[a-z0-9\-]   => [a-z0-9-]
[a-z0-9_ \-] => [a-z0-9_ -]
[ _\-]       => [ _-]
[\.0-9]      => [.0-9]
[0-9\.]      => [0-9.]
[ \-]        => [ -]
[_ \-]       => [_ -]
[_\.]        => [_.]
[_ \.]       => [_ .]
[ _\-\.]     => [ _.-]

#7640 (comment)

@sanchezzzhak sanchezzzhak enabled auto-merge (squash) November 5, 2024 05:48
@sanchezzzhak sanchezzzhak merged commit bedd142 into matomo-org:master Nov 5, 2024
15 checks passed
@liviuconcioiu liviuconcioiu deleted the regex branch November 5, 2024 09:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants