Skip to content
This repository has been archived by the owner on Oct 14, 2024. It is now read-only.

Misc sarif updates #80

Merged
merged 12 commits into from
Dec 5, 2023
Merged

Misc sarif updates #80

merged 12 commits into from
Dec 5, 2023

Conversation

ZacSweers
Copy link
Collaborator

@ZacSweers ZacSweers commented Dec 5, 2023

This does a few cleanups around Sarif handling

  • Add a level option to baseline merging
  • Add some utilities for comparing results at a shallow level
  • Mark merged lint baselines as suppressed
  • Implement a new ApplyBaselinesToSarifs CLI for updating or merging sarif results based on a given baseline. This has two modes (see their docs) for use with either a baseline of suppressed issues (i.e. detekt/lint baseline files) or a baseline of the base branch that it's updating from. This will mark the final output with baselineData and suppressions accordingly.

@ZacSweers ZacSweers requested review from valeraz and linhpha December 5, 2023 22:33
@ZacSweers ZacSweers marked this pull request as ready for review December 5, 2023 22:33
@ZacSweers ZacSweers added this pull request to the merge queue Dec 5, 2023
Merged via the queue into main with commit 0729231 Dec 5, 2023
2 checks passed
@ZacSweers ZacSweers deleted the z/sarifUpdates branch December 5, 2023 22:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants