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

Regex: corrected regex pattern to match valid annotation #156

Open

hiteshbedre wants to merge 2 commits into 1Password:main from hiteshbedre:main

hiteshbedre commented May 2, 2023

Corrected dot(.) matches by adding escape pattern.

Invalid operator string such as operator11passwordiio\/some\text, operator$1password0io\/some\text are also getting matched with current pattern.

Added fix so it will matches only desired string such as operator.1password.io\/some\text

created playground to test pattern: https://regex101.com/r/r5fUXv/1

hiteshbedre added 2 commits

May 2, 2023 17:22


          regex: updated annotation RegEx pattern

a228ea9


          regex: update RegEx defined under suite

4bcbaef

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet