Skip to content
This repository was archived by the owner on Jan 12, 2026. It is now read-only.

[BitSail][Docs] Update License.#239

Merged
BlockLiu merged 9 commits into
bytedance:masterfrom
BlockLiu:refactor-license
Dec 15, 2022
Merged

[BitSail][Docs] Update License.#239
BlockLiu merged 9 commits into
bytedance:masterfrom
BlockLiu:refactor-license

Conversation

@BlockLiu

Copy link
Copy Markdown
Collaborator

Signed-off-by: Liu Peng

Pre-Checklist

Note: Please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests.

Purpose

Some description about what this PR wants to do.

Approaches

Change old ASF license to standard Apache 2.0 license.

Related Issues

Close #236

New Behavior (screenshots if needed)

N/A

@BlockLiu BlockLiu changed the title Update License. [BitSail][Docs] Update License. Dec 13, 2022

@healchow healchow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@BlockLiu BlockLiu merged commit bd83d9d into bytedance:master Dec 15, 2022
@BlockLiu BlockLiu deleted the refactor-license branch December 15, 2022 10:40
Comment thread LICENSE
identification within third-party archives.

Copyright [yyyy] [name of copyright owner]
Copyright 2022 Bytedance Ltd. and/or its affiliates.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@BlockLiu
This is the License header template that needs to be applied to the source code.
We are not supposed to add the copyright information here.
Could you review the change?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Just LICENSE file right?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed, see #342.
Thanks for your comment!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Please don't use ASF Customered License header

3 participants