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

Issue #142 Improve performance of match any word --- Updated matchAnyWords and MatchAllWords to work with curried string o… #152

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leftfields
Copy link

Updated matchAnyWords and MatchAllWords to work with curried string operators opposed to regexps -- seems faster and more reliable on larger inputs. Also added new test cases

…perators opposed to regexps -- seems faster and more reliable on larger inputs. Also added new test cases
@daedalus28
Copy link
Contributor

Thanks for the contribution! We're a bit swamped here trying to recover after the hurricane (our devs are mostly all based in south florida), but we'll get to this soon!

@sadasant
Copy link
Contributor

@leftfields hi! Sorry for the super late response. Do you mind updating this branch? Once it's up to date, the code will be auto-formatted with prettier. After that we'll most likely merge it!

@daedalus28
Copy link
Contributor

@sadasant I don't think prettier will run for PRs from non-contirbutors 🤔

@sadasant
Copy link
Contributor

@daedalus28 I can make a copy of this changes, or what would be the better approach?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants