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

[v1.9.x] Remove the "copyright by contributors" line in source files - #20493

Merged
szha merged 8 commits into
apache:v1.9.xfrom
TristonC:issue-20475-item5
Aug 7, 2021
Merged

[v1.9.x] Remove the "copyright by contributors" line in source files#20493
szha merged 8 commits into
apache:v1.9.xfrom
TristonC:issue-20475-item5

Conversation

@TristonC

@TristonC TristonC commented Aug 6, 2021

Copy link
Copy Markdown
Contributor

Description

This PR is the action item 5, for the issue 20475. It removes the "Copyright by Contributors" line in the ./src, ./include, ./tools and 3rdparty/mshadow directories and their sub-directories.

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Remove the "Copyright by Contributors" line in the ./src direcotry and its sub-directories.
  • Remove the "Copyright by Contributors" line in the ./include direcotry and its sub-directories.
  • Remove the "Copyright by Contributors" line in the ./tools direcotry and its sub-directories.
  • Remove the "Copyright by Contributors" line in the ./3rdparty/mshadow direcotry and its sub-directories.

Comments

@mxnet-bot

Copy link
Copy Markdown

Hey @TristonC , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [edge, sanity, centos-cpu, unix-gpu, clang, windows-gpu, website, unix-cpu, miscellaneous, windows-cpu, centos-gpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@mseth10 mseth10 added the pr-awaiting-testing PR is reviewed and waiting CI build and test label Aug 6, 2021
@TristonC TristonC changed the title [License] Remove the "copyright by contributors" line in source files [v1.9.x] Remove the "copyright by contributors" line in source files Aug 6, 2021
@TristonC
TristonC requested a review from szha as a code owner August 6, 2021 20:23
Comment thread src/kvstore/comm.h Outdated
Comment thread src/kvstore/gpu_topology.h Outdated
Comment thread src/operator/numpy/np_einsum_op.cc
@mseth10 mseth10 added pr-awaiting-review PR is waiting for code review and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Aug 6, 2021
@szha
szha merged commit eefb371 into apache:v1.9.x Aug 7, 2021
@barry-jin barry-jin mentioned this pull request Sep 24, 2021
4 tasks
barry-jin added a commit that referenced this pull request Sep 28, 2021
* remove copyright lines

* fix

* update

* update stes_op.cc

* fix

Co-authored-by: Triston <triston.cao@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr-awaiting-review PR is waiting for code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants