Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

*: Add copyright owner in go files #364

Merged
merged 2 commits into from
Feb 2, 2018
Merged

*: Add copyright owner in go files #364

merged 2 commits into from
Feb 2, 2018

Conversation

gaocegege
Copy link
Member

@gaocegege gaocegege commented Feb 1, 2018

Close #266

Signed-off-by: Ce Gao [email protected]


This change is Reviewable

@k8s-ci-robot
Copy link

Hi @gaocegege. Thanks for your PR.

I'm waiting for a kubernetes or tensorflow member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@coveralls
Copy link

coveralls commented Feb 1, 2018

Coverage Status

Coverage remained the same at 31.746% when pulling 9e8825c on gaocegege:copyright into 330eb92 on tensorflow:master.

@ewilderj
Copy link

ewilderj commented Feb 1, 2018

Please revise to Copyright The Kubeflow Authors, vs Google.

@jlewi
Copy link
Contributor

jlewi commented Feb 1, 2018

I apologize for giving incorrect and misleading information about the author.

@gaocegege
Copy link
Member Author

@ewilderj I filed the PR before I saw the comment 😄 And it could be frozen until we have a conclusion about the copyright owner.

@jlewi Never mind 😃

@ewilderj
Copy link

ewilderj commented Feb 2, 2018

@gaocegege thanks for your work on this!

@gaocegege
Copy link
Member Author

I have updated the owner to The Kubeflow Authors, PTAL

Copy link
Member

@ScorpioCPH ScorpioCPH left a comment

Choose a reason for hiding this comment

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

Thanks for this work!
For Copyright comments, I prefer to /* */ style rather than //.

@gaocegege
Copy link
Member Author

@jlewi jlewi merged commit 4676740 into kubeflow:master Feb 2, 2018
@gaocegege gaocegege deleted the copyright branch March 6, 2018 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants