Skip to content
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

fix: removeNoise function not working correctly #557

Closed
wants to merge 1 commit into from
Closed

fix: removeNoise function not working correctly #557

wants to merge 1 commit into from

Conversation

dubisdev
Copy link
Contributor

@dubisdev dubisdev commented May 18, 2022

Hi,
I am currently developing some cerebro plugins and have noticed that the current function that removes words from plugin names is not working correctly. This is the case of cerebro-todoist, whose name is modified appearing only "Doist". I've slightly modified the regex for each usage (name and description) so there should no longer be a problem with names like "cerebro-toggle-..." appearing as "ggle".

The new regex removes words like 'cerebro-' and 'cerebro-plugin-' from the plugins name but not affecting to the rest of the name.
image

Before

image

After
image

@dubisdev dubisdev marked this pull request as ready for review May 18, 2022 11:00
@dubisdev dubisdev mentioned this pull request Jun 22, 2022
9 tasks
@dubisdev dubisdev closed this Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant