-
Notifications
You must be signed in to change notification settings - Fork 4
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
Switch to LGPLv2.1+ #41
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
|
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.
I agree to the relicensing under LGPLv2.1+.
I'll try to contact @rodpollet . I am blocking until we have their agreement.
@orbeckst I've just emailed @rodpollet Also I'm going to ignore the windows failures for now - I believe these will fix themselves once we release a new version of MDAnalysis. |
@fiona-naughton @ianmkenney if you can approve the relicense this would be great. |
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.
Block until we have @rodpollet 's agreement to the license change.
Just noticed that the license is also mentioned in |
Well spotted—thanks for fixing!Am 11/22/24 um 21:27 schrieb Fiona Naughton ***@***.***>:
Just noticed that the license is also mentioned in README.md under the "Copyright" section, so it also needs to be updated there (pathsimanaysis and mdahole2 both also have the old license info in the README, I can make a quick some PRs for them)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
This is done thanks. |
@orbeckst we have the agreement, are you ok to approve? |
Whilst the original code contribution has been accepted as LGPLv2.1+, we have to relicense this MDAKit.
PR Checklist