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

fix: remove use of rego keyword #1661

Merged
merged 1 commit into from
Jul 30, 2024
Merged

fix: remove use of rego keyword #1661

merged 1 commit into from
Jul 30, 2024

Conversation

didroe
Copy link
Contributor

@didroe didroe commented Jul 30, 2024

Description

Renames contains to array_contains in rego code.

contains is a Rego keyword and this causes an error in later versions. See #1656

Checklist

If this is your first time contributing please sign the CLA

  • I've added test coverage that shows my fix or feature works as expected.
  • I've updated or added documentation if required.

@didroe didroe requested a review from elsapet July 30, 2024 11:29
elsapet
elsapet approved these changes Jul 30, 2024
View reviewed changes
Copy link
Contributor

@elsapet elsapet left a comment

Choose a reason for hiding this comment

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

🙌

@didroe didroe merged commit 5ed94a9 into main Jul 30, 2024
8 checks passed
@didroe didroe deleted the chore/remove-use-of-rego-keyword branch July 30, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elsapet elsapet elsapet 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.
2 participants
@didroe @elsapet