Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update documentation for nested first-match rules - #1323

#1323
Merged
jnthntatum merged 5 commits into
cel-expr:mastercel-expr/cel-go:masterfrom
jnthntatum:compile-policy-nesting-2jnthntatum/cel-go:compile-policy-nesting-2Copy head branch name to clipboard
May 21, 2026
Merged

Update documentation for nested first-match rules#1323
jnthntatum merged 5 commits into
cel-expr:mastercel-expr/cel-go:masterfrom
jnthntatum:compile-policy-nesting-2jnthntatum/cel-go:compile-policy-nesting-2Copy head branch name to clipboard

Conversation

@jnthntatum

Copy link
Copy Markdown
Collaborator

First match rules have an implicit unwrap behavior if they are unconditional. This wasn't documented and behaved inconsistently depending on where the rule was in the policy tree.

First match rules have an implicit unwrap behavior if they are
unconditional. This wasn't documented and behaved inconsistently
depending on where the rule was in the policy tree.
@jnthntatum jnthntatum assigned l46kok and unassigned l46kok May 19, 2026
@jnthntatum
jnthntatum requested a review from l46kok May 19, 2026 00:27
@jnthntatum
jnthntatum marked this pull request as ready for review May 19, 2026 20:17
l46kok
l46kok previously approved these changes May 19, 2026
Comment thread policy/README.md Outdated
Comment thread policy/helper_test.go
Comment thread policy/README.md Outdated
Comment thread policy/README.md Outdated
@jnthntatum
jnthntatum merged commit 3c645ee into cel-expr:master May 21, 2026
7 checks passed
copybara-service Bot pushed a commit to cel-expr/cel-policy that referenced this pull request Jun 1, 2026
copybara-service Bot pushed a commit to cel-expr/cel-java that referenced this pull request Jun 1, 2026
copybara-service Bot pushed a commit to cel-expr/cel-policy that referenced this pull request Jun 1, 2026
copybara-service Bot pushed a commit to cel-expr/cel-java that referenced this pull request Jun 1, 2026
copybara-service Bot pushed a commit to cel-expr/cel-java that referenced this pull request Jun 2, 2026
copybara-service Bot pushed a commit to cel-expr/cel-java that referenced this pull request Jun 2, 2026
copybara-service Bot pushed a commit to cel-expr/cel-java that referenced this pull request Jun 2, 2026
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.