Skip to content

Conversation

leongyh
Copy link

@leongyh leongyh commented Feb 17, 2023

include args to trivy config file

include args to trivy config file
@CLAassistant
Copy link

CLAassistant commented Feb 17, 2023

CLA assistant check
All committers have signed the CLA.

@simar7
Copy link
Member

simar7 commented Apr 18, 2023

thanks @leongyh - can we add a small test case which can exercise this functionality? You take this as a starting point to write a new case https://github.com/aquasecurity/trivy-action/blob/master/test/test.bats#L77-L83

@hermanwh
Copy link
Contributor

hermanwh commented May 9, 2023

I tried to provide an attempt at the suggested test here: #231

Feel free to include such a test in your own PR if desired, and have this merged instead of mine.
I suppose it would be ideal with additional tests concerning parameter precedence etc.

@simar7
Copy link
Member

simar7 commented May 31, 2023

Closed in favour of #231

@simar7 simar7 closed this May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants