-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
Fix for #52 - Eliminate Duplicate Licenses #54
Fix for #52 - Eliminate Duplicate Licenses #54
Conversation
@iankerr Since I merged your other PR, can you fix your conflicts? |
…ows#53) * make any links to licenses clickable in preformatted text * fix failing test * rename currentProjectName to CurrentLicenseName
Hmmm... I merged in your recent changes, and now the build is failing. Error: "Could not create an instance of Tooling API implementation using the specified Gradle distribution 'https://services.gradle.org/distributions/gradle-3.4-bin.zip'" Any idea what's wrong? |
This reverts commit 9a8a3a5.
This reverts commit 9a8a3a5.
Sorry. I merged without fixing the build. Do you mind pushing this back up so we can merge it? |
I've re-created the pull request: #57 ...hopefully that's what you're asking for? |
This is is a proposed fix for issue #52
I changed the HtmlReport class a bit to lookup licenses in the license map by url and name, and group them all together in the report.
Also:
-adds elements for all licenses (including "no license) - not just the ones in the license map
-updates tests to reflect my changes
-adds more names/urls for the Apache 2.0 license
-adds the CC0 1.0 Universal license to the map