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.

Sign up for GitHub

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

Jump to bottom

[Lib] graph builtins #83

Closed
3 tasks done
anakrish opened this issue Dec 31, 2023 · 0 comments · Fixed by #86
Closed
3 tasks done

[Lib] graph builtins #83

anakrish opened this issue Dec 31, 2023 · 0 comments · Fixed by #86
Labels
lib Library function

Comments

@anakrish
Copy link
Collaborator

anakrish commented Dec 31, 2023 •
edited
Loading

Note: graph.reachable_paths has bugs in upstream OPA:

Get cargo test -r --test opa -- reachable walkbuiltin to pass.
@anakrish anakrish added the lib Library function label Dec 31, 2023
This was referenced Dec 31, 2023
Merged
Merged
anakrish added a commit that referenced this issue Jan 1, 2024
@anakrish
Handle walk builtin as a loop expression (#86)
b470c3f 
The walk builtin generates values and implicitly creates a loop over the values.
Hoist walk calls as loops and handle them. Also handle cases where return value
is bound to an extra parameter.

Closes #83
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lib Library function
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Handle walk builtin as a loop expression anakrish/regorus
1 participant
@anakrish