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

Define MIT license at composer.json #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

phansys
Copy link

@phansys phansys commented Dec 5, 2024

Currently, the package definition does not include the license type.
Since this is a open source package, the license declaration is important to expose how this package can be consumed.

I used the MIT license because this is already used at knplabs/gaufrette.

By instance, I'm using a license checker to confirm that all my dependencies (direct or transitive) comply with my licensing constraints, but the absence of the license declaration in this package blocks its installation.

Please. let me know if you require further context or a different license type, like Unlicense.

BTW, as the same situation seem the same in majority of packages under the gaufrette/ vendor, I'd be glad to provide PRs for the remaining packages if this proposal is accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant