Skip to content
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: iac documentation url in outputs #1881

Merged
merged 1 commit into from May 6, 2021

Conversation

almog27
Copy link

@almog27 almog27 commented May 5, 2021

What does this PR do?

This PR adds documentation links to the IaC --json and --sarif outputs.
For each issue, we will present a link for the relevant public documentation page.

How should this be manually tested?

Run IaC test (new flow / experimental) with json/sarif output

Screenshots

Json output
Screen Shot 2021-05-05 at 9 53 18

Sarif output
Screen Shot 2021-05-05 at 9 53 53

Regular output - not affected
Screen Shot 2021-05-05 at 9 54 49

@almog27 almog27 requested a review from a team May 5, 2021 06:56
@almog27 almog27 self-assigned this May 5, 2021
@almog27 almog27 requested a review from rontalx May 5, 2021 06:56
@github-actions
Copy link
Contributor

github-actions bot commented May 5, 2021

Messages
📖

This PR will not trigger a new version. It doesn't include any commit message with feat or fix.

Generated by 🚫 dangerJS against be34e5c

@almog27 almog27 merged commit 9c6dcfd into master May 6, 2021
@almog27 almog27 deleted the refactor/iac-documentation-url-in-outputs branch May 6, 2021 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants