Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@klag
Copy link
Member

@klag klag commented Aug 5, 2025

I added findAll in order to find all the occurencies of a string into a target string.
the existing api find, just finds only the first occurence

@fmontesi
Copy link
Member

fmontesi commented Aug 7, 2025

To understand: I guess you could use match and a regex with a star (*) to do the same thing, right?

@klag
Copy link
Member Author

klag commented Aug 7, 2025

match is usually exploited for just matching a string into another

@fmontesi
Copy link
Member

Ah, I see, the key difference is that in a match I have to use capturing groups in the request, whereas here capturing (to return the results) is always done implicitly, I just have to pass a normal regex. Correct?
If so, I think we're good to go.

@fmontesi fmontesi merged commit 0b84ad2 into master Aug 18, 2025
6 checks passed
@fmontesi fmontesi deleted the enhancement/findAll branch August 18, 2025 04:19
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.

3 participants