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

enable strict compilation rules in policies #776

Closed
boranx opened this issue Feb 17, 2023 · 2 comments · Fixed by #798
Closed

enable strict compilation rules in policies #776

boranx opened this issue Feb 17, 2023 · 2 comments · Fixed by #798
Labels
enhancement New feature or request

Comments

@boranx
Copy link
Member

boranx commented Feb 17, 2023

hey team

There's a nice tweak https://www.openpolicyagent.org/docs/latest/strict/ that allows the user to impose strict compilation rules in a given policy such as checking unused variables or unused imports. I think this might be synced down to conftest too so the users would have a similar experience compared to opa
I think the implementation of it would be pretty straight-forward because it's already an option for NewCompiler() if it is tainted with WithStrict(true) and could be manageable thru a flag

Looking forward to hearing your thoughts on that, thanks all in advance

@jalseth jalseth added the enhancement New feature or request label Feb 19, 2023
@jalseth
Copy link
Member

jalseth commented Feb 23, 2023

Seems like a good idea to me. I'd be open to a PR if you have the time.

@boranx
Copy link
Member Author

boranx commented Feb 27, 2023

ack, I'll propose a PR if I find some cycles

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants