Skip to content

apply licence to MongoDB Inc. #196

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
May 21, 2020

Conversation

MaBeuLux88
Copy link
Contributor

According to the previous version of this file, this is how to apply Apache 2.0 to a project:

APPENDIX: How to apply the Apache License to your work.

  To apply the Apache License to your work, attach the following	
  boilerplate notice, with the fields enclosed by brackets "[]"	
  replaced with your own identifying information. (Don't include	
  the brackets!)  The text should be enclosed in the appropriate	
  comment syntax for the file format. We also recommend that a	
  file or class name and description of purpose be included on the	
  same "printed page" as the copyright notice for easier	
  identification within third-party archives.

identification within third-party archives.

Copyright [yyyy] [name of copyright owner]
Copyright 2020 MongoDB Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

@MaBeuLux88 did you mean to delete the whole of this LICENSE file, or just replace line 190 with Copyright 2020 MongoDB Inc.?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I meant it. The licence is already in the link.

You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0

I guess it's also possible to keep it like this for example:
https://github.com/moby/moby/blob/master/LICENSE

Let me know which one you prefer, I can amend the PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

We prefer to keep this file in our repos. I think we just forgot to correct the Copyright placeholder.

Copy link
Contributor Author

@MaBeuLux88 MaBeuLux88 May 20, 2020

Choose a reason for hiding this comment

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

OK done.
I re-inserted the LICENCE text and removed the paragraph that explains how to apply the licence which isn't needed anymore.

To apply the Apache License to your work...

@MaBeuLux88 MaBeuLux88 force-pushed the apply-licence-correctly branch from 3f38b9b to 643eb02 Compare May 20, 2020 12:31
@MaBeuLux88 MaBeuLux88 force-pushed the apply-licence-correctly branch from 643eb02 to 1e63015 Compare May 20, 2020 12:31
@lrlna lrlna merged commit e091f04 into mongodb-js:master May 21, 2020
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.

2 participants