search

iacsecurity / tool-compare

MIT License
278 stars 60 forks source link

Update test case results for tfsec with new container #9

Closed yi2020 closed 3 years ago

owenrumney commented 3 years ago

Yup, looks like it's my bad, I missed a docker container name https://github.com/iacsecurity/tool-compare/blob/2b84a946176b5de7f7f7dbaa2bfd532139a262dd/run_all_tools.sh#L37 -

in the docker run it still has liamg/tfsec. Want me to raise a new PR or can you add it to this one?

yi2020 commented 3 years ago

@owenrumney I got it.

yi2020 commented 3 years ago

@owenrumney using the tfsec/tfsec container there's a bunch of test cases that now are caught by tfsec that weren't earlier, good job :)

Anything else before I merge this PR?

owenrumney commented 3 years ago

@yi2020 nope, all good

yi2020 commented 3 years ago

Closes #8