-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Acataldi/10779 fix schema registry lint errors #12011
Acataldi/10779 fix schema registry lint errors #12011
Conversation
Thank you for your contribution abc516! We will review the pull request and get back to you soon. |
@@ -45,7 +45,7 @@ const testGroup = "azsdk_js_test_group"; | |||
const testSchemaIds = [ | |||
"{773E17BE-793E-40B0-98F1-0A6EA3C11895}", | |||
"{DC7EF290-CDB1-4245-8EE8-3DD52965866E}" | |||
].map((x) => x.replace(/[\{\-\}]/g, "")); | |||
].map((x) => x.replace(/[{\-}]/g, "")); |
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.
@abc516 Can you share which linting rule failure is this change fixing?
@nguerrera, Do you recall what exactly are we trying to replace here and why?
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.
@ramya-rao-a @nguerrera this fixes this rule: https://eslint.org/docs/rules/no-useless-escape .
I believe it just subs out the '{', '-', and '}' chars in the guid strings with blank spaces. Note the test below from https://regexr.com , using the new expression :
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.
Yes, that's what it does and this is fine. 😄
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.
Thanks Nick
Thanks @abc516! |
Thanks, @abc516 ! |
fixes #10779