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

fixes #6206; using cuboid rather than cube #6691

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

kd-labs
Copy link
Contributor

@kd-labs kd-labs commented Apr 10, 2024

Why the changes in this PR are needed?

The change involves using a cuboid rather than a cube to explain concepts of sets and composite values in policy-language section of documentation.

What are the changes in this PR?

The changes are in a single file i.e. policy-language.md

Notes to assist PR review:

Signed-off-by: kunal.das <kd.kunaldas92@gmail.com>
Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for openpolicyagent ready!

Name Link
🔨 Latest commit fabbe7a
🔍 Latest deploy log https://app.netlify.com/sites/openpolicyagent/deploys/66168e989f108b000879b435
😎 Deploy Preview https://deploy-preview-6691--openpolicyagent.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kd-labs
Copy link
Contributor Author

kd-labs commented Apr 10, 2024

snap of the composite and sets section containing the changes

image

image

Copy link
Contributor

@srenatus srenatus left a comment

Choose a reason for hiding this comment

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

Thanks!

@ashutosh-narkar ashutosh-narkar merged commit ae636de into open-policy-agent:main Apr 10, 2024
26 checks passed
@kd-labs kd-labs deleted the issue-6206 branch April 10, 2024 17:22
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.

3 participants