-
Notifications
You must be signed in to change notification settings - Fork 444
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.
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
[cisco_ios]: escape hyphen inside regex char list #9578
[cisco_ios]: escape hyphen inside regex char list #9578
Conversation
🚀 Benchmarks reportTo see the full report comment with |
042f979
to
3cea41e
Compare
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
💚 Build Succeeded
History
|
Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices) |
@jrmolin @dwhyrock @taylor-swanson apparently if I don't escape the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a pretty interesting find. Nice work!
I'm actually surprised that this makes a difference. I knew about moving the hyphen to the end (which is what I typically do), but I would've assume that escaping it would've compiled to the same result. |
Proposed commit message
After tackling this issue, a respective PR was created to
elastic-package
to mainstream the detection of similar warnings. Thus this PR fixes the following similar to the former issue regex-related warning forcisco_ios
Checklist
changelog.yml
file.Author's Checklist
N/A
How to test this PR locally
Related issues
N/A
Screenshots
N/A