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
Added Epidemico and emails to the readme #450
Conversation
Copyright 2015 Kitware Inc. | ||
|
||
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://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. | ||
Licensed under the Apache License, Version 2.0 (the "License"); you may not use |
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.
The license file in the repository contains the 3-clause BSD license. You should update that as well as the license line in package.json.
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.
@jbeezley thanks...for catching this. I will update the license file and the license file in package.json.
f77aeb9
to
7c40b68
Compare
db2f2ee
to
1a8d5b0
Compare
@jbeezley @dcjohnston @nicopresto please review |
same "printed page" as the copyright notice for easier | ||
identification within third-party archives. | ||
|
||
Copyright [yyyy] [name of copyright owner] |
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.
Fill in the blanks.
Other than the copyright statement, it LGTM. |
I realized that too but then when I looked at Girder LICENSE, it has APPENDIX as well. See here https://github.com/Kitware/girder/blob/master/LICENSE so I left it thinking its custom to keep the entire text. Since the agreement ends before that so I will take it out. |
Added Epidemico and emails to the readme
@dcjohnston @nicopresto @jbeezley please review. Any suggestion would be useful.