space-ros / process_sarif

Tools to process and aggregate SARIF output from static analysis.
Apache License 2.0
6 stars 2 forks source link

Allow for missing ruleId. #9

Closed Ronoman closed 10 months ago

Ronoman commented 1 year ago

This will be introduced if this ament_lint PR is merged. More commits will be added on this PR to support better de-duplication, depending on discussion in that PR's comments.

ivanperez-keera commented 1 year ago

I just posted a comment in the associated PR in ament_lint to try to move things along.

Bckempa commented 11 months ago

Provisionally nominating for inclusion in the next release, we're waiting on upstream for the the fix to ament_lint but that doesn't seem to preclude getting this one-liner merged.