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 Apache 2.0 LICENSE #51

Closed
wants to merge 2 commits into from
Closed

Conversation

chartpath
Copy link
Contributor

@chartpath chartpath commented Oct 28, 2023

Currently it would be a copyright violation for community users to modify this repo because it is missing a license.

For comparison, the other popular Supabase GenAI community repo https://github.com/supabase-community/vercel-ai-chatbot is already using the Apache 2.0 license.

@chartpath
Copy link
Contributor Author

BTW I just noticed Supabase uses Apache 2.0 for their main release so I figured that would be consistent in terms of choices. There seem to be a few other community projects that use MIT as well.

@chartpath
Copy link
Contributor Author

For more reference on the rationale of why not having a license can be restrictive to contributors, see https://opensource.stackexchange.com/questions/1720/what-can-i-assume-if-a-publicly-published-project-has-no-license

@chartpath chartpath closed this Nov 14, 2023
@gregnr gregnr mentioned this pull request Nov 15, 2023
@gregnr gregnr reopened this Nov 15, 2023
@gregnr
Copy link
Contributor

gregnr commented Nov 15, 2023

Hey @chartpath, sorry for the delay on this. This is a good call - we've added the Apache 2.0 license now. I mistakenly thought this thread was an issue and not a PR, so merged the change in a separate PR #52. Sorry about that.

Thanks for the suggestion!

@gregnr gregnr closed this Nov 15, 2023
@chartpath
Copy link
Contributor Author

@gregnr Thank you! This will allow me to make a contribution now :)

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