Skip to content
This repository was archived by the owner on Oct 10, 2024. It is now read-only.

Conversation

@AllanRegush
Copy link
Contributor

closes #460

This PR adds Left quote and right quote to the special characters regex. Removing the ability to escape warnings from the bot.

Screenshot_20210304_090257

Copy link

@github-actions github-actions bot left a 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.

Copy link
Member

@rsrinath14 rsrinath14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@AllanRegush
Copy link
Contributor Author

Thanks for reviewing @srinath1412001

function stripSpecialCharacters(str: string) {
// match special symbols and replace with ' '
str = str.replace(/[.,/#!$%?&*;:{}=\-_'"~()]/g, ' ');
str = str.replace(/[.,/#!$%?&*;:{}=\-_'"~()]/g, ' ');
Copy link
Member

@Vyvy-vi Vyvy-vi Mar 7, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM-!

@AllanRegush AllanRegush merged commit 76c6607 into EddieHubCommunity:develop Mar 7, 2021
@AllanRegush AllanRegush deleted the allan-issue-460 branch March 7, 2021 19:21
@AllanRegush
Copy link
Contributor Author

Thanks for reviewing everyone.

@eddiejaoude
Copy link
Member

Great collaboration EddieHubbers 🤓 (is that how you spell it?)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inclined Quotation marks are not detected

5 participants