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

Create LICENSE #46

Merged
merged 1 commit into from
Feb 9, 2018
Merged

Create LICENSE #46

merged 1 commit into from
Feb 9, 2018

Conversation

pckroon
Copy link
Member

@pckroon pckroon commented Feb 8, 2018

Today I feel like Apache.
This is pending approval from Tsjerk and Siewert-Jan. In addition, we need to add the following header to all files:

   Copyright 2018 University of Groningen

   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
   You may obtain a copy of the License at

       http:https://www.apache.org/licenses/LICENSE-2.0

   Unless required by applicable law or agreed to in writing, software
   distributed under the License is distributed on an "AS IS" BASIS,
   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   See the License for the specific language governing permissions and
   limitations under the License.

Today I feel like Apache.
@pckroon pckroon added the WIP label Feb 8, 2018
@pckroon
Copy link
Member Author

pckroon commented Feb 8, 2018

In addition, we might also need to look at a contributor license agreement (CLA) where people agree in advance that we (actually people in Marrink-Lab) can change the license at a later date --- as long as it remains open source.
https://github.com/cla-assistant/cla-assistant

@jbarnoud jbarnoud added this to the Release 1.0 milestone Feb 8, 2018
@jbarnoud jbarnoud requested a review from Tsjerk February 8, 2018 12:29
@pckroon
Copy link
Member Author

pckroon commented Feb 8, 2018

Tsjerk and SJ agree. I suggest we merge this for now, but leave it open so we can add the license headers later on.

@pckroon pckroon removed the WIP label Feb 8, 2018
Copy link
Collaborator

@Tsjerk Tsjerk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed and approved

@Tsjerk Tsjerk merged commit 1d4c4b2 into master Feb 9, 2018
@Tsjerk Tsjerk deleted the add-license-1 branch February 9, 2018 08:49
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.

None yet

3 participants