Skip to content

Allow merging sarifs with different schemas #220

@matejdro

Description

@matejdro

Recently, Detekt updated its schema URL, but Android Lint still has the old URL. Even though both output sarif 2.1.0, this makes them unmergeable:

require(schema == other.schema) { "Cannot merge sarifs with different schemas: '$schema' or '${other.schema}'" }

Considering that we already check the version, is the schema check even necessary? If yes, could we add a flag to disable it?

I can make a PR for this.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions