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 to the autoloader #6

Merged
merged 2 commits into from
Feb 5, 2015
Merged

Add MIT License to the autoloader #6

merged 2 commits into from
Feb 5, 2015

Conversation

sylvainraye
Copy link
Member

As provided by the original author of this class, please add the MIT license to prevent problem by using it.
Here is the original link to the gist of the author: https://gist.github.com/jwage/221634

Sylvain Rayé added 2 commits January 31, 2015 11:48
As provided by the original author of this class, please add the MIT license to prevent problem by using it.
Here is the original link to the gist of the author: https://gist.github.com/jwage/221634
daim2k5 added a commit that referenced this pull request Feb 5, 2015
Add MIT License to the autoloader
@daim2k5 daim2k5 merged commit bfc33f0 into magento-hackathon:master Feb 5, 2015
@Flyingmana
Copy link
Member

conflicts with the licence noted in composer.json

@sylvainraye
Copy link
Member Author

In composer you cannot set two licenses. how could it be resolved? Removing the GPL license? or keeping the most constraining ?

@sylvainraye
Copy link
Member Author

Sorry I was wrong we can set multiple license: "licenses": ["MIT", "GPL"],

@Flyingmana
Copy link
Member

btw. MIT is fine for me

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.

3 participants