-
-
Notifications
You must be signed in to change notification settings - Fork 135
fix(words): matching special unicode qoutes #460 #463
fix(words): matching special unicode qoutes #460 #463
Conversation
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.
It's great having you contribute to this project
Welcome to the community 🤓If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.
rsrinath14
left a comment
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!
|
Thanks for reviewing @srinath1412001 |
| function stripSpecialCharacters(str: string) { | ||
| // match special symbols and replace with ' ' | ||
| str = str.replace(/[.,/#!$%?&*;:{}=\-_'"~()]/g, ' '); | ||
| str = str.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.
LGTM-!
|
Thanks for reviewing everyone. |
|
Great collaboration EddieHubbers 🤓 (is that how you spell it?) |
closes #460
This PR adds Left quote
“and right quote”to the special characters regex. Removing the ability to escape warnings from the bot.