8000 feat(942480): move to regex assembly by fzipi · Pull Request #4032 · coreruleset/coreruleset · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(942480): move to regex assembly #4032

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

feat(942480): move to regex assembly #4032

wants to merge 2 commits into from

Conversation

fzipi
Copy link
Member
@fzipi fzipi commented Mar 3, 2025

what

  • move 942480 to full regex assembly

why

  • compat
  • updating format

@github-actions GitHub Actions
Copy link
Contributor
github-actions bot commented Mar 3, 2025 •
edited
Loading

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

@fzipi
feat(942480): move to regex assembly
79192e1 
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
@fzipi fzipi force-pushed the regex/942480-move-regex-assembly branch from 9b438d7 to 79192e1 Compare March 3, 2025 12:44
@fzipi fzipi requested a review from theseion March 24, 2025 12:09
theseion
theseion requested changes Mar 25, 2025
View reviewed changes
@fzipi
fix: apply code review suggestion
7c57ff0 
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
@fzipi fzipi force-pushed the regex/942480-move-regex-assembly branch from a8b85d2 to 7c57ff0 Compare March 26, 2025 13:26
@github-actions github-actions bot added the Stale label Apr 26, 2025
@Xhoenix Xhoenix removed the Stale label Apr 27, 2025
@github-actions github-actions bot added the Stale label May 28, 2025
@Xhoenix Xhoenix removed the Stale label May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theseion theseion theseion requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
3532 @fzipi @theseion @Xhoenix
0