Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Update license, add CHANGELOG, make changes to fit OT, add CODEOWNERS #4

Merged
merged 3 commits into from
Nov 5, 2019

Conversation

lzchen
Copy link
Contributor

@lzchen lzchen commented Nov 5, 2019

According to CELA, it is safe to change the Apache license to the MIT license when moving from OT to Microsoft repo because all the changes were made by Microsoft employees.

  1. Update license
  2. Add CHANGELOG for release
  3. Add some new changes from OT repo
  4. Add to CODEOWNERS file

@lzchen lzchen requested review from reyang and hectorhdzg November 5, 2019 21:49
@@ -0,0 +1,5 @@
# Code owners file.
Copy link
Member

Choose a reason for hiding this comment

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

We don't have Code owners file in any ApplicationInsights repo, why adding it here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is not an ApplicationInsights repo.

Copy link
Member

Choose a reason for hiding this comment

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

I know that, I want to know why you are adding this here?, what is the purpose of this file?, how can this be useful? ,etc. It feels this is irrelevant in a Microsoft owned repo but maybe I'm wrong

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The purpose of this file is so that any new issues will have the aliases in the CODEOWNERS file automatically tagged to review the PRs. It is not a REPO specific concept but a GITHUB concept.

## Unreleased

## 0.1a.0
Released 2019-11-05
Copy link
Member

Choose a reason for hiding this comment

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

Did you released this today?

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'm planning on releasing today.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually on second thought, I should change this to unreleased until I actually release it.

@lzchen lzchen merged commit 6f78ea0 into microsoft:master Nov 5, 2019
@lzchen lzchen deleted the license branch November 5, 2019 22:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants