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

fomatters: Add SARIF formatter #704

Merged
merged 1 commit into from Sep 13, 2021
Merged

fomatters: Add SARIF formatter #704

merged 1 commit into from Sep 13, 2021

Conversation

m-ildefons
Copy link
Collaborator

Add formatter for SARIF output format. SARIF is the static analysis
results interchange format, a Microsoft/OASIS standard format for static
analysis tools.
See https://github.com/microsoft/sarif-tutorials
and https://docs.oasis-open.org/sarif/sarif/v2.1.0/sarif-v2.1.0.html

Break long lines in app/Main.hs.

How to verify it

hadolint -f sarif Dockerfile should generate a valid SARIF report for that dockerfile. Tools like jsonschemavalidator can be used to validate the output against the schema spec.

Example dockerfile with output:

FROM debian:latest

WORKDIR ./foobar

RUN echo $foo

screengrab2

Add formatter for SARIF output format. SARIF is the static analysis
results interchange format, a Microsoft standard format for static
analysis tools.
See https://github.com/microsoft/sarif-tutorials

Break long lines in `app/Main.hs`.
@lorenzo
Copy link
Member

lorenzo commented Sep 13, 2021

thanks!

@lorenzo lorenzo merged commit 3b1de5a into hadolint:master Sep 13, 2021
@jbergstroem
Copy link
Contributor

Thanks for this! Been looking at doing it myself

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.

None yet

3 participants