FSharp.Analyzers.SDK icon indicating copy to clipboard operation
FSharp.Analyzers.SDK copied to clipboard

Sarif merging

Open nojaf opened this issue 3 months ago • 2 comments

Is your feature request related to a problem? Please describe.

GitHub Actions no longer accepts multiple sarif files for upload. This is biting my in every project where I use analyzers.

Describe the solution you'd like Can we add a MSBuild target that accepts an array/list of projects and invokes the cli in one go?

<Target Name="AnalyzeFSharpProjects" DependsOnTargets="_AnalyzeFSharpProject" />

Describe alternatives you've considered It is possible to merge these sarif files, see https://github.com/fsprojects/fantomas/blob/1a6f2dbff5bf25b4fd9fbd6315a8b8cf8bb005f8/sarif.fsx#L231

But that requires everyone to deal with this themselves. So maybe a breaking change in the MSBuild targets is the simpler thing.

Additional context I don't have the right answer here, happy to discuss.

nojaf avatar Sep 01 '25 11:09 nojaf

Just looking around, I see things like:

I wonder if these would be sufficient (haven't tried).

TheAngryByrd avatar Sep 01 '25 15:09 TheAngryByrd

Thanks, I'll try those and maybe just document what worked well over here!

nojaf avatar Sep 02 '25 07:09 nojaf