-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add outputs to show result in pull requests #31
Comments
Hey @robermp we're trying to do something very similar - where did you insert the above code to get the output you wanted? Do you have a PR against this repo that you could make available? |
Hi I insert the code just before these lines inside entrypoint.sh file: Executing terrascanecho "Executing terrascan as follows:" I cant' do a PR because i'm not contributor |
This worked for me. Thank you! I was also able to use unsplash/comment-on-pr@v1.3.0 to return output to our CI system. Very cool! |
Like this, to push a comment back to the PR:
|
Hi @cesar-rodriguez, i generate the PR: |
The stdout of the action is empty, so to see the results in PR some lines are needed:
With this the in the PR the output of the terrascan can be read like this:
Using stdout:
![2021-10-21 17_13_12-add key rotation by robermp · Pull Request #29 · grupoasv_alz_aws_network and 15](https://user-images.githubusercontent.com/37579601/138306857-c409899c-0dec-4cf0-b3c5-b2da45f270ce.png)
Using log output:
![2021-10-21 17_12_22-add key rotation by robermp · Pull Request #29 · grupoasv_alz_aws_network and 15](https://user-images.githubusercontent.com/37579601/138306798-ff0db4a1-8780-4247-9cbe-d1e3bf265ad5.png)
The text was updated successfully, but these errors were encountered: