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

Try to match file contents to known license copy #4

Merged
merged 7 commits into from
Feb 23, 2015

Conversation

joeyAghion
Copy link
Contributor

This is perhaps a little loose, but can be helpful. It tries to match known license texts to the likely README or LICENSE files of a gem. In one project with 188 dependencies, this permitted 39 additional licenses to be found.

Obviously, it's possible that a gem includes the text from a common license and appends its own modifications, so this isn't rock-solid.

No tests yet.

@dblock
Copy link
Owner

dblock commented Feb 20, 2015

This is nice. Will merge with tests.

  • I think common_licenses should go into lib/licenses/examples or something like that.
  • Rubocop! :)

@dblock
Copy link
Owner

dblock commented Feb 21, 2015

I would merge with pending tests too ;)

@dblock
Copy link
Owner

dblock commented Feb 23, 2015

I'm going to merge this. Don't hesitate to write tests though if you feel like it ;)

dblock added a commit that referenced this pull request Feb 23, 2015
Try to match file contents to known license copy
@dblock dblock merged commit 8180164 into dblock:master Feb 23, 2015
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.

2 participants