aboutcode-org / aboutcode-org/scancode-toolkit

Matched text diagnostics should be in a new attribute

Open
#3,426 9 comments 0 reactions 0 assignees View on GitHub
bug good first issue license scan
Dominant language
Python
Stars
2.6k
Forks
791
Avg merge
1d 12h
Merged PRs (30d)
5

Description

Currently when we have `--matched-text-diagnostics` enabled we add this diagnostics directly in the `matched_text` attribute for the match. This is problematic as this `matched_text` is important in many downstream cases like generating attribution, showing diffs etc, and so this should be in a new attribute `matched_text_diagnostics`.
There would be a downside of increased file size for the result file btw.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.