-
Notifications
You must be signed in to change notification settings - Fork 67
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
Implement Match #6
Comments
Punting on this for the moment, the specification is rather complicated |
Hey, so today it raised an error when he finds a "Match" directive pattern, maybe can we just pass through it without raise anything ? |
@kevinburke would you consider a PR that made it an option to skip |
@kevinburke ping (about @cv question). I think it's still valid and needs to be handled. Thanks. |
Yes I agree. While #45 (#38) does help a little, we loose a lot of functionality that is provided by the |
Would that be ok ? |
I'm worried I'm not smart enough to implement
Match exec
properly; it seems like a security risk.The text was updated successfully, but these errors were encountered: