Skip to content

Conversation

@liangyuanpeng
Copy link
Contributor

@liangyuanpeng liangyuanpeng commented Apr 21, 2021

changed:

  1. add licenses of apache for runtime module
  2. replace com.webank to org.apache fro files of other than java files

fixed #295

@kezhenxu94
Copy link
Member

@liangyuanpeng did you add the license headers with license-eye? I think you'd better also add it as GitHub Action to automatically check and review future pull requests, it's a cool feature to ease the reviewers from reviewing the license and the contributors to add license headers, it works like this apache/skywalking#6716 (review) , if you have problem using it, I'm glad to help

Copy link
Contributor

@xwm1992 xwm1992 left a comment

Choose a reason for hiding this comment

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

nice job

Copy link
Contributor

@keranbingaa keranbingaa left a comment

Choose a reason for hiding this comment

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

LGTM

@xwm1992 xwm1992 merged commit afa9f95 into apache:develop Apr 22, 2021
@liangyuanpeng
Copy link
Contributor Author

@kezhenxu94 Yes, I just use container for license-eye now and i will try it from github action. Will consider contributing to eventmesh.

xwm1992 added a commit that referenced this pull request Aug 4, 2022
[ISSUE #295] add licenses of apache for runtime module
close #295
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.

Lack of licenses in each source file under the eventmesh-runtime module

4 participants