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 MIT license compatibility badge to the README #4612

Closed
wants to merge 1 commit into from
Closed

Add MIT license compatibility badge to the README #4612

wants to merge 1 commit into from

Conversation

kmewhort
Copy link

@kmewhort kmewhort commented Sep 3, 2016

I just ran Git.legal's license compatibility scan on all the included gems. All checkout as compatible with the MIT license. There was just one LGPL license, which checks-out as truly being used a library, so no problems there.

The badge here gives a quick indicator that the license is MIT, and that the libraries used are compliant with that.

Copy link
Contributor

@varyonic varyonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to see the following changes added: git mv LICENCE MIT-LICENCE I checked the content of the LICENCE text file is an MIT licence. Add a line at the bottom of the README, MIT Licence. Copyright (c) Greg Bell, VersaPay Corporation

@varyonic
Copy link
Contributor

For the curious, this is the activeadmin page on Git.legal

@varyonic
Copy link
Contributor

@kmewhort I would make these changes for you, but you did not branch from master. Please re-submit with my requested changes if interested, but I will have to close this.

@varyonic varyonic closed this Jan 29, 2017
@timoschilling
Copy link
Member

@varyonic what was the problem with the branch?

@varyonic
Copy link
Contributor

He wanted to merge from his master. Is that possible? If he renames his master then GitHub would close this anyway, no?

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.

None yet

3 participants