Skip to content

Conversation

@touchweb-vincent
Copy link
Contributor

@touchweb-vincent touchweb-vincent commented Nov 12, 2025

Hello,

I guess there are two tiny typos here.

Our French student in BTS CIEL (Cybersécurité, Informatique et Réseaux, Électronique) will thank you for this change.

This list is missing mod, but I really dislike the idea of putting a three-letter alphanumeric sequence in a PL1 regex.

Thank you

@github-actions
Copy link
Contributor

📊 Quantitative test results for language: eng, year: 2023, size: 10K, paranoia level: 1:
🚀 Quantitative testing did not detect new false positives

@touchweb-vincent touchweb-vincent changed the title fix(942151): wrong function name fix(942151,942152): wrong function name Nov 12, 2025
@touchweb-vincent touchweb-vincent changed the title fix(942151,942152): wrong function name fix(942151,942152): wrong functions names Nov 12, 2025
Copy link
Member

@fzipi fzipi left a comment

Choose a reason for hiding this comment

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

Nicely spotted!

@fzipi fzipi added this pull request to the merge queue Nov 13, 2025
Merged via the queue into coreruleset:main with commit 0d9355b Nov 13, 2025
13 checks passed
@touchweb-vincent touchweb-vincent deleted the patch-10 branch November 13, 2025 15:08
@fzipi fzipi mentioned this pull request Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants