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

docs: Add v0 import example to v0-compatibility #8

Merged

Conversation

charlieegan3
Copy link

This shows another way to use two versions of rego in the same app.

Copy link

@johanfylling johanfylling left a comment

Choose a reason for hiding this comment

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

👍

docs/content/v0-compatibility.md Outdated Show resolved Hide resolved
"fmt"

"github.com/open-policy-agent/opa/rego"
// v1rego "github.com/open-policy-agent/opa/v1/rego"
Copy link
Owner

Choose a reason for hiding this comment

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

Did you intend to keep this commented out line for informational purposes or something?

Copy link
Author

Choose a reason for hiding this comment

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

Yeah it was intentional but I've made that clearer in d6db4ac

Signed-off-by: Charlie Egan <charlie@styra.com>
@ashutosh-narkar ashutosh-narkar merged commit b0a9eaf into ashutosh-narkar:1.0-docs Dec 18, 2024
23 checks passed
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