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

Build weighted graph #369

Merged
merged 10 commits into from
Oct 16, 2024
Merged

Build weighted graph #369

merged 10 commits into from
Oct 16, 2024

Conversation

yissellokta
Copy link
Contributor

Description

Build the weighted graph based on the language graph, providing tuple cycle detection, model cycle detection, and type mismatch detection

References

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@yissellokta yissellokta requested review from a team as code owners October 10, 2024 14:13
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Oct 10, 2024 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

elbuo8
elbuo8 reviewed Oct 10, 2024
View reviewed changes
pkg/go/.golangci.yaml Outdated Show resolved Hide resolved
@elbuo8 elbuo8 enabled auto-merge October 16, 2024 13:29
@elbuo8 elbuo8 added this pull request to the merge queue Oct 16, 2024
Merged via the queue into main with commit d8e33c3 Oct 16, 2024
10 checks passed
@elbuo8 elbuo8 deleted the feat-complete-weighted-graph branch October 16, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@senojj senojj senojj left review comments

@miparnisari miparnisari miparnisari left review comments

@adriantam adriantam adriantam approved these changes

@elbuo8 elbuo8 elbuo8 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yissellokta @elbuo8 @miparnisari @senojj @adriantam