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

Fix license #30

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Fix license #30

merged 1 commit into from
Nov 8, 2023

Conversation

blinxen
Copy link
Contributor

@blinxen blinxen commented Nov 7, 2023

When I added the license file in my last PR, I failed to notice that this repository is a fork. MIT licensed forks must keep the original license text in the fork. While you can re-license the fork and just keep the old license file around, not changing it would make the life of a distro packager (me :D) a lot more easier. I hope that is possible.

@postsolar
Copy link
Collaborator

postsolar commented Nov 7, 2023

In postsolar/tree-sitter-purescript#5 I expressed the idea of separating the licenses for the legacy Haskell code and new PureScript code so as to make the new code have a more permissive license and let future forkers not bother about any of this at least for the PS part. Absolutely not a hill to die on for me though, I'd mostly just want it to get resolved in whichever convenient way.
EDIT: looks like it's a bad idea. Opting for the convenient resolution then.

@Maskhjarna
Copy link
Owner

I skimmed your original PR and mistook the included license for MIT. Sorry! LGTM

@Maskhjarna Maskhjarna merged commit ae21a27 into Maskhjarna:main Nov 8, 2023
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