scorecard
scorecard copied to clipboard
:seedling: Combine hasLicenseFile and hasLicenseFileAtTopDir probes
What kind of change does this PR introduce?
probe cleanup
- [X] PR title follows the guidelines defined in our pull request documentation
What is the current behavior?
There are 3 license probes, but 2 of them (hasLicenseFile and hasLicenseFileAtTopDir) always exhibit the same positive/negative behavior so there are effectively only 2 license probes.
What is the new behavior (if this is a feature change)?**
-
hasLicenseFileAtTopDiris deleted - ensure license findings use the license file locations
- finding logging is simplified using the standard
checker.LogFindingsfunction- since
hasFSFOrOSIApprovedLicensecan sometimes returnOutcomeNotApplicable, this converts one message to aDEBUGmessage. But its identical to theWARNmessage already produced byhasLicenseFile.
- since
before:
"details": [
"Info: project does not have a license file",
"Info: project does not have a license file"
],
after:
"details": [
"Warn: project does not have a license file",
"Debug: project does not have a license file"
],
- [X] Tests for the changes have been added (for bug fixes/features)
Tests modified at least
Which issue(s) this PR fixes
Fixes #3590
Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their application as a result of this pull request.)
Reduced duplicate logging messages related to License details.
Codecov Report
Merging #3955 (9118bb8) into main (be15709) will increase coverage by
1.87%. The diff coverage is82.35%.
Additional details and impacted files
@@ Coverage Diff @@
## main #3955 +/- ##
==========================================
+ Coverage 66.77% 68.65% +1.87%
==========================================
Files 227 226 -1
Lines 16249 16147 -102
==========================================
+ Hits 10851 11085 +234
+ Misses 4729 4360 -369
- Partials 669 702 +33
/scdiff generate License