Skip to content

Feat: added removeWords - #1

Merged
Samarth2190 merged 1 commit into
Samarth2190:mainfrom
JohnCervantes:feat/remove-words
May 28, 2025
Merged

Feat: added removeWords#1
Samarth2190 merged 1 commit into
Samarth2190:mainfrom
JohnCervantes:feat/remove-words

Conversation

@JohnCervantes

@JohnCervantes JohnCervantes commented May 27, 2025

Copy link
Copy Markdown
Contributor

Key Changes:

  - Implemented removeWords and added it on transformations.js file
  examples:
  stringzy.removeWords('The quick brown fox jumps over the lazy dog', ['quick', 'lazy'])); //The brown fox jumps over the dog
  stringzy.removeWords('The quick brown fox jumps over the lazy dog', 'lazy')); //The quick brown fox jumps over the dog

  - refactored index (maybe this is not really needed? can revert)

@Samarth2190

Copy link
Copy Markdown
Owner

Great Work. Just a one change.
There needs to some error handling for parameters if the user passes them as integer or objects.
I will just modify it a bit.
This is really a good addition

@Samarth2190
Samarth2190 merged commit 46cceba into Samarth2190:main May 28, 2025
@Samarth2190

Samarth2190 commented May 28, 2025

Copy link
Copy Markdown
Owner

I think we should stick to the older format of index.js, where we were exporting individual functions too. This will allow user to export only the needed function

I'll be making some changes there too.

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.

2 participants