-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
refactor(misconf): Remove unused options #7896
Conversation
Signed-off-by: Simar <[email protected]>
cc8c0ee
to
b5963ca
Compare
@simar7 Does that mean we should now only support |
|
Yes, there's even a comment about this https://github.com/aquasecurity/trivy/blame/797b36fbad90b8e7f04e16e2cf08d6bdc0255ac7/pkg/iac/rego/scanner.go#L387 Actually I just realized we didn't issue an announcement for this change. I will prepare that first. Discussion here: #7961 |
@simar7 As we discussed earlier, will support for |
Yes, is there something that I've missed in this PR? |
Description
These options aren't often used due to reasons described in the disucussion and can be deprecated.
Related issues
Related discussion
Checklist