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

ast: Fixing erroneous missing value assignment error #6368

Conversation

johanfylling
Copy link
Contributor

for rules/functions that only has value assignment for else but not "primary" head

Fixes: #6364

@ashutosh-narkar ashutosh-narkar force-pushed the fix/rego-v1_erroneous_missing_value_assignment_err branch from 3b5dff5 to c800441 Compare October 31, 2023 16:01
Copy link
Member

@ashutosh-narkar ashutosh-narkar left a comment

Choose a reason for hiding this comment

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

LGTM

for rules/functions that only has value assignment for `else` but not "primary" head

Fixes: open-policy-agent#6364
Signed-off-by: Johan Fylling <[email protected]>
@ashutosh-narkar ashutosh-narkar force-pushed the fix/rego-v1_erroneous_missing_value_assignment_err branch from c800441 to bdb70de Compare October 31, 2023 16:28
@johanfylling johanfylling merged commit 0309239 into open-policy-agent:main Nov 1, 2023
24 checks passed
@johanfylling johanfylling deleted the fix/rego-v1_erroneous_missing_value_assignment_err branch November 1, 2023 14:04
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.

rego.v1: rego_parse_error: rule must have value assignment and/or body declaration
2 participants