Skip to content

Update license text to match official 3-clause-BSD #142

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

Merged
merged 1 commit into from
Jun 16, 2023

Conversation

skypanther
Copy link
Contributor

This minimally updates your license to make it easier to use your library. Because of your changes from the standard text and conventions, we were forced to go through an internal legal review by our corporate lawyer. I hope to save others from the same.

  • Rename the license file to LICENSE to match convention and make it easier to find (our lawyer thought your project was unlicensed since the file wasn't named "right")
  • Replace your modified 3-clause BSD license with the standard text taken from https://opensource.org/licenses/BSD-3-Clause.

You are the "copyright holder" so replacing with "the author" makes no meaningful change. But, it meant our lawyer insisted on carefully reviewing the rest of the text to be sure you hadn't changed the license (from a legal perspective). Also, our lawyer's feedback was that "All Rights Reserved" is not needed under copyright law. I left it out but I don't think it matters a ton. https://en.wikipedia.org/wiki/All_rights_reserved

@coveralls
Copy link

coveralls commented Dec 9, 2022

Coverage Status

Coverage decreased (-5.7%) to 60.146% when pulling 7f1829b on skypanther:standardlicense into 714df3c on stefankoegl:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-5.7%) to 60.146% when pulling 7f1829b on skypanther:standardlicense into 714df3c on stefankoegl:master.

@stefankoegl stefankoegl merged commit 33562b0 into stefankoegl:master Jun 16, 2023
@stefankoegl
Copy link
Owner

Thanks!

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