Skip to content

Conversation

@vthinkxie
Copy link
Contributor

@vthinkxie vthinkxie commented Jan 16, 2020

What is this PR for?

Fix miss apache license header

What type of PR is it?

[Bug Fix]

What is the Jira issue?

[ZEPPELIN-4557]

How should this be tested?

https://travis-ci.org/vthinkxie/zeppelin/builds/637915032

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@zjffdu
Copy link
Contributor

zjffdu commented Jan 16, 2020

Thanks for contribution @vthinkxie Could you update the PR description and paste the travis CI link ?

@vthinkxie
Copy link
Contributor Author

updated cc @zjffdu

@zjffdu
Copy link
Contributor

zjffdu commented Jan 17, 2020

Thanks @vthinkxie Although some tests are still failed, I think there're due to other issues.
Will merge it soon

@asfgit asfgit closed this in 8f5059c Jan 17, 2020
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.

2 participants