-
Notifications
You must be signed in to change notification settings - Fork 134
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
Migrated the license to MIT #84
Conversation
Created this for #56 |
I also bumped up the version to 2.0.x as I consider a license change a potential breaking / major change. |
I already regretted it, so I undid the change. |
It's been a while, what's the status of this PR? |
Life came in between, just wait. |
Hey @punker76 & @hardcodet can you take a look at this, so we can merge it? |
Hey @Lakritzator and @hardcodet I saw the discussion about this in #56. This is a really great project, and pulling in this license change would definitely make my life easier. Any chance of that happening? Thanks, and good luck! |
LICENSE
Outdated
|
||
Preamble | ||
Copyright (c) Dapplo and Contributors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to have been copy/pasted from Dapplo ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Damn, I was hoping you would oversee it, so I have all rights... no just kidding.
Let me correct that...
Any news? |
I'm a bit interested in this license change as well due CPOL not being on a corporate's acceptable use list. Are there any stoppers from this being merged? @hardcodet @punker76 |
@Lakritzator @hardcodet LGTM |
Hi, any chance that this PR can be added to the main base instead of waiting on a 2.0.0 milestone, that has no clear ETA? |
Also removed .NET 5.0 support as this is EOL.
Closes #56