getappmap / scanner

Code scanning, linting, assertions and alerts.
Other
0 stars 0 forks source link

Deduplicate findings #90

Closed dividedmind closed 2 years ago

dividedmind commented 2 years ago

Also, deduplicate metadata.

I suggest reviewing commit-by-commit. Finding hashes are currently unchanged and there's more to be shaved off, but this still manages to cut down the upload size significantly. (For example with my saleor dataset it reduces 13605 total findings to 129.)

kgilpin commented 2 years ago

Thanks I will take a look. Please investigate the build failure.

kgilpin commented 2 years ago

By the way in findings.dedup.json, summary.numFindings = 91, but there are only 7 findings after de-duplication.

appland-release commented 2 years ago

:tada: This PR is included in version 1.38.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: