Prepare docs for OPA 1.0 compatibility #1328
Merged
+239
−400
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
anderseknert
force-pushed
the
opa1.0-docs
branch
2 times, most recently
from
f8e036d to
0d5e9a5
Compare
| configuration. Note that this has lower precedence than Rego versions set by other means, and should not be considered | ||
| as anything but a convenience for testing. | ||
|
|
||
| ## Rules disabled with OPA 1.0 |
There was a problem hiding this comment.
Some rules are labelled Notice: Rule made obsolete by OPA 1.0, others Notice: Rule disabled with OPA 1.0 - is there a subtle difference I'm missing?
There was a problem hiding this comment.
Theimport-shadows-import rule should have the "obsolete" header, and I'll fix that. The only other one that has "rule disabled" header is deprecated-builtin. That rule isn't really made obsolete — it's just that we currently don't have any deprecated built-ins in OPA 1.0. But that may very well change, and the rule will be enabled once that happens.
- Remove `import rego.v1` from examples - Add new page covering Regal and OPA 1.0 - Update logic of any rules obsolete in 1.0 to make sure they're automatically disabled when 1.0 is targeted, and continue to work when older versions are linted - Update a few tests to remove `import rego.v1` - Many minor fixes related to this theme Signed-off-by: Anders Eknert <[email protected]>
anderseknert
force-pushed
the
opa1.0-docs
branch
from
0d5e9a5 to
c8c5f9d
Compare
anderseknert
added a commit
that referenced
this pull request
Jan 13, 2025
- Remove `import rego.v1` from examples - Add new page covering Regal and OPA 1.0 - Update logic of any rules obsolete in 1.0 to make sure they're automatically disabled when 1.0 is targeted, and continue to work when older versions are linted - Update a few tests to remove `import rego.v1` - Many minor fixes related to this theme Signed-off-by: Anders Eknert <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
import rego.v1from examplesimport rego.v1