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

Implement regex builtins #57

Merged
merged 1 commit into from
Sep 9, 2021
Merged

Implement regex builtins #57

merged 1 commit into from
Sep 9, 2021

Conversation

ereslibre
Copy link
Member

Related: #38

Implements the following builtins:

  • regex.split
  • regex.template_match
  • regex.find_n

From this group, regex.globs_match is left unimplemented due to the
ratio effort/benefit and lack of real world consumers.

@ereslibre ereslibre added the area/rego Issue related with Rego/OPA/Gatekeeper integration label Sep 1, 2021
@ereslibre ereslibre force-pushed the regex-builtins branch 7 times, most recently from 84929e5 to 500408e Compare September 2, 2021 08:42
- `regex.split`
- `regex.template_match`
- `regex.find_n`

From this group, `regex.globs_match` is left unimplemented due to the
ratio effort/benefit and lack of real world consumers.
Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, that wasn't trivial. 👏

@ereslibre ereslibre merged commit bfc5d5e into opa Sep 9, 2021
@ereslibre ereslibre deleted the regex-builtins branch September 9, 2021 16:44
@ereslibre ereslibre mentioned this pull request Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rego Issue related with Rego/OPA/Gatekeeper integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants