Skip to content

Commit

Permalink
Add SarifParser
Browse files Browse the repository at this point in the history
Fix #8
  • Loading branch information
Jérémie Bertrand committed Sep 1, 2016
1 parent 59b3e01 commit 124bdf9
Show file tree
Hide file tree
Showing 25 changed files with 2,432 additions and 114 deletions.
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -194,4 +194,4 @@ tools/OpenCover/
tools/xunit.runner.console/
tools/coveralls.io/
tools/ReportGenerator
src/NVika/static-analysis.sarif.json
src/.vs/config/applicationhost.config
7 changes: 5 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,16 @@ It is possible to process several reports at the same time: `NVika parsereport r
## Analysis tools
### Supported
- [InspectCode](https://chocolatey.org/packages/resharper-clt): example of usage `inspectcode /o="inspectcodereport.xml" "Vika.sln"`
- Analyzers producing [SARIF](http://sarifweb.azurewebsites.net) format, like Roslyn analyzers: for those you need to add an `ErrorLog` node in your `csproj` containing the path of the report; See [NVika.csproj](https://github.com/laedit/vika/blob/master/src/NVika/NVika.csproj)

### To come
- [FxCop](https://github.com/laedit/vika/issues/6)
- [CodeCracker](https://github.com/laedit/vika/issues/8)
- [StyleCop](https://github.com/laedit/vika/issues/7)
- NDepend?
- DupFinder (if someone wants it reaaaally bad)
- [Mono.Gendarme](https://github.com/laedit/vika/issues/16)
- [CPD](https://github.com/laedit/vika/issues/27)
- [CPPCheck](https://github.com/laedit/vika/issues/26)

## Build servers
### Supported
Expand All @@ -51,7 +54,7 @@ It is possible to process several reports at the same time: `NVika parsereport r
- [MyGet](https://github.com/laedit/vika/issues/5)?

I really wondering if there is any value to supporting these three, because there doesn't support to add build message like AppVeyor but only log message.
And they support custom HTML report, so an xsl transformation is enough.
And they support custom HTML report, so an xslt transformation is enough.

## What it will be
A website will be added for displaying a nice and shiny aggregated report from all source to a dedicated page for each GitHub project.
Expand Down
3 changes: 3 additions & 0 deletions appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ artifacts:
- path: build\NVika.exe.CodeAnalysisLog.xml
name: NVika.exe.CodeAnalysisLog.xml

- path: build\static-analysis.sarif.json
name: static-analysis.sarif.json

- path: artifacts\inspectcodereport.xml

- path: src\NVika\static-analysis.sarif.json
Expand Down
Loading

0 comments on commit 124bdf9

Please sign in to comment.