-
Notifications
You must be signed in to change notification settings - Fork 307
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 parse combine info to index #940
docs: add parse combine info to index #940
Conversation
0049d77
to
149ef08
Compare
Signed-off-by: Amber Beasley <[email protected]>
149ef08
to
90ebe3e
Compare
Signed-off-by: Amber Beasley <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR and keeping our docs up to date! A couple of small changes.
app: hello-kubernetes | ||
``` | ||
|
||
> To use the builtin functions, you need to include `github.com/open-policy-agent/conftest/builtins` in your Go file import section if you did not include it elsewhere for default go commands configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels like it belongs in a "developer docs" section rather than unit test section. We don't really have a page for that yet, would you mind starting one?
|
||
**combine.yaml** | ||
|
||
```yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you reduce the size of this example to only include the deployment that matches the rule's test?
Hi @amber-beasley-liatrio friendly ping. Are you still working on this? |
Closing this due to inactivity. Feel free to reopen if you pick this up again. |
hoping to provide information that was not trivial/obvious to those newer to rego and conftest