-
Notifications
You must be signed in to change notification settings - Fork 55
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
Added support for sarif reports #108
Conversation
@armadi1809 Looking good so far - just needs some tests! |
@armadi1809 Looks like this fork is getting a bit out of date. Please resolve the merge conflicts |
As usual, I'm likely to drop into the discussion, but it's the second time in a week I read about serif, and I have no idea what it could be. The other project were I saw it was golangci-lint I think. Could anyone explain me what it is, what it would allow to this project? Thanks, Christophe |
It started as a standard format for outputting static analysis results but now its use has expanded to general test output like JUnit. |
OK for thanks for replying me. I might have a look at it |
} | ||
|
||
jsonBytes = append(jsonBytes, '\n') | ||
fmt.Print(string(jsonBytes)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line looks like a left behind debug used for dev purpose
Work was never completed and #166 replaces it |
Still need to add tests but keeping this as draft in case someone notices something wrong. This should be ready soon.