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

Update OpenSearch LicenseHeaders to use the short version #29

Merged

Conversation

peterzhuamazon
Copy link
Member

@peterzhuamazon peterzhuamazon commented Aug 12, 2021

Signed-off-by: Peter Zhu zhujiaxi@amazon.com

Description

Update OpenSearch LicenseHeaders to use the short version.

  • It removes the Amazon related info and change to OpenSearch Contributors for copyrights.
  • It also changed to a short version which is cleaner and still has enough information.

Issues Resolved

#21

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon changed the title Update OpenSearch LicenseHeaders Update OpenSearch LicenseHeaders to use the short version Aug 12, 2021
@dblock dblock mentioned this pull request Aug 12, 2021
1 task
@dblock
Copy link
Member

dblock commented Aug 12, 2021

Is the short version the preferred version, @hyandell?

@hyandell
Copy link
Member

Yes, that's the preferred version.

In the text however, I would say this:

New files in your code contributions should contain the following license header. If you are modifying existing files with license headers, or including new files that already have license headers, do not remove or modify them without guidance.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon
Copy link
Member Author

@hyandell Please check again I updated the text based on your suggestions.

@peterzhuamazon
Copy link
Member Author

Thanks @hyandell for approval @dblock please merge it. 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