nexB / scancode-analyzer

scancode-results-analyzer
4 stars 2 forks source link

[RFC] Drop resource-level license detection issue reports in favour of codebase-level summary, with unique license detection issues #51

Open AyanSinhaMahapatra opened 3 years ago

AyanSinhaMahapatra commented 3 years ago

Presently with https://github.com/nexB/scancode-results-analyzer/pull/44, there are two attributes added,

codebase_attributes : "license_detection_issues_summary"

resource_attributes : "license_detection_issues"

where the resource_attribute represents a file-view example, where every file has license detection issues potentially, and the codebase_attribute is an issue-view example, where we go through issues (unique issues, with all their occurances) one by one, instead of files.

The issue-view makes more sense in terms of review and resolving, and as @pombredanne suggests even dropping the resource_attribute entirely, and just have the codebase_attribute as an issue-view, makes sense.