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

Add license header #7379

Merged
merged 4 commits into from
Aug 8, 2017
Merged

Add license header #7379

merged 4 commits into from
Aug 8, 2017

Conversation

mli
Copy link
Contributor

@mli mli commented Aug 8, 2017

Added ASF license header to all files except for

  • submodules
  • R-package (not apache2)
  • src/operator/mkl/ (licensed to intel)
  • src/operator/contrib/ctc_include/ (licensed to nvidia)

Also add a CI job to test if new added files have a proper header

/*!
* Copyright (c) 2015 by Contributors
* \file softmax_activation.cu
* \brief
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the original header needs to be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

technically it is hard, need to write more codes to merge two headers. i'd like to put it into the future works.

@lxn2
Copy link
Member

lxn2 commented Aug 8, 2017

Can we merge?

@piiswrong piiswrong merged commit 251ae71 into apache:master Aug 8, 2017
Guneet-Dhillon pushed a commit to Guneet-Dhillon/mxnet that referenced this pull request Sep 13, 2017
* add

* add .py and ci

* fix pylint

* update
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.

3 participants