Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Relax the SarifReader tool name matching logic to use startswith inst… #88

Merged
merged 1 commit into from
Aug 7, 2024
Merged

Relax the SarifReader tool name matching logic to use startswith inst… #88

merged 1 commit into from
Aug 7, 2024

Commits on Aug 7, 2024

  1. Relax the SarifReader tool name matching logic to use startswith inst… 

    …ead of equals so versions of the same tool will match. For example, Semgrep OSS and Semgrep PRO use the same file format but their names are slightly different. So now the SemgrepSarifReader class names the tool simply 'Semgrep' and SarifReader uses startswith() so both versions of the same tool match with one reader. And added a test case to test this.
    @davewichers
    davewichers committed Aug 7, 2024
    Configuration menu
    Copy the full SHA
    072a679 View commit details
    Browse the repository at this point in the history