Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new licenses and rules #1282

Merged
merged 11 commits into from
Nov 14, 2018
Merged

Add new licenses and rules #1282

merged 11 commits into from
Nov 14, 2018

Conversation

pombredanne
Copy link
Member

@pombredanne pombredanne commented Nov 12, 2018

This a set of new and improved licenses and license detection rules based on and following a bug report by @jonasob for #1281 and other such as @DennisClark #1269

 * add new Apache detection rule to handle overlap case found in
   SignalR such that no unknown is returned

 * add new Apache detection rule to handle non-standard notice found
   in SignalR to return a higher score

Reported-by: Jonas Öberg @jonasob
Signed-off-by: Philippe Ombredanne <[email protected]>
 * ms-net-library-2018-11 newly found in SignalR at same older URL
   https://www.microsoft.com/web/webpi/eula/net_library_eula_ENU.htm
   This is a modification of the previous ms-net-library license but
   is not versioned by Microsft and found exactly at the same URL
 * also rename and update older version rules

Reported-by: Jonas Öberg @jonasob
Signed-off-by: Philippe Ombredanne <[email protected]>
 * replaced by bsd-source-cod license

Signed-off-by: Philippe Ombredanne <[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
 * improve detection based on SignalR scans with new rules and/or
   adding a stored relevance if needed

Reported-by: Jonas Öberg @jonasob
Signed-off-by: Philippe Ombredanne <[email protected]>
 * Adssign unknown-license-reference to rules that were unknown by
   mistake

Reported-by: Jonas Öberg @jonasob
Signed-off-by: Philippe Ombredanne <[email protected]>
 * improve detection based on SignalR repos scans with new rules and/or
   adding or updating the rule "relevance" as needed

Reported-by: Jonas Öberg @jonasob
Signed-off-by: Philippe Ombredanne <[email protected]>
 * improve detection primarily for license file references and mostly
   for things from Microsoft

Reported-by: Jonas Öberg @jonasob
Signed-off-by: Philippe Ombredanne <[email protected]>
@pombredanne pombredanne force-pushed the new-licenses-and-rules branch from 81f8091 to 0b971e1 Compare November 13, 2018 12:07
@codecov
Copy link

codecov bot commented Nov 13, 2018

Codecov Report

Merging #1282 into develop will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1282   +/-   ##
========================================
  Coverage    84.12%   84.12%           
========================================
  Files          118      118           
  Lines        13037    13037           
========================================
  Hits         10967    10967           
  Misses        2070     2070
Impacted Files Coverage Δ
src/licensedcode/index.py 81.92% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6f16d1...9f5b169. Read the comment docs.

@pombredanne
Copy link
Member Author

All green... merging!

@pombredanne pombredanne merged commit d4d10ac into develop Nov 14, 2018
@pombredanne pombredanne deleted the new-licenses-and-rules branch November 14, 2018 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant