Skip to content

Commit

Permalink
Get canonical version of holder nexB#2972
Browse files Browse the repository at this point in the history
    * Update expected test results
    * TODO: consider not converting common company names to a canonical form

Signed-off-by: Jono Yang <jyang@nexb.com>
  • Loading branch information
JonoYang authored and KevinJi22 committed Jun 14, 2022
1 parent 56a9d5e commit 98cf292
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/summarycode/summarizer.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
from cluecode.copyrights import CopyrightDetector
from packagedcode.utils import combine_expressions
from packagedcode import models
from summarycode.copyright_tallies import canonical_holder
from summarycode.score import compute_license_score
from summarycode.score import get_field_values_from_codebase_resources
from summarycode.score import unique
Expand Down Expand Up @@ -167,7 +168,7 @@ def get_declared_holders(codebase, holders_tallies):
codebase, 'holders', key_files_only=True
)
entry_by_key_file_holders = {
fingerprints.generate(entry['holder']): entry
fingerprints.generate(canonical_holder(entry['holder'])): entry
for entry in key_file_holders
if entry['holder']
}
Expand Down

0 comments on commit 98cf292

Please sign in to comment.