-
Notifications
You must be signed in to change notification settings - Fork 16
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
string.matches #2
Comments
Hi @jchonde, Unfortunately I don't have access to this repo any more as I've left the company where the work was made, and they seem now unresponsive to my pull requests. I never implemented regular expression serialisation, but I suggest you could patch this with a PR by looking into the code around these lines, and running a check of the type for each argument. Alternately I would suggest using the |
Are you thinking of forking or moving the code base to a place where you can keep maintaining it, if you are interested? |
Outlined in #10 |
I tried executing the regular expression passed from BE using |
Hello,
You did an amazing job with that ast, thanks for that.
I'm struguling using the
string.matches
from yup.I should be able to write something like that if I understood correctly the tests:
But it seems that regexp serialization is not working properly or is not implemented.
Have you any suggestions, ideas ?
The text was updated successfully, but these errors were encountered: