You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by minahlee <gi...@git.apache.org> on 2018/05/04 15:13:12 UTC
[GitHub] zeppelin pull request #2957: [ZEPPELIN-3441] Fix license check failure in r
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/2957
[ZEPPELIN-3441] Fix license check failure in r
### What is this PR for?
rat plugin for r interpreter fail, seems to be related with #2089
### What type of PR is it?
Hot Fix
### What is the Jira issue?
[ZEPPELIN-3441](https://issues.apache.org/jira/browse/ZEPPELIN-3441)
### How should this be tested?
Run `mvn verify -Pr` should pass
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/minahlee/zeppelin hotfix/ratR
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2957.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2957
----
commit c323c4a8b6d21bb7158d2321e8f5e26cc33319e3
Author: Mina Lee <mi...@...>
Date: 2018-05-04T14:28:04Z
Fix rat check exclude directory for r
commit 1d4ae6c91cdd031ccd785e28497bfaf217e4087d
Author: Mina Lee <mi...@...>
Date: 2018-05-04T14:29:29Z
Add rat check for r in travis
----
---
[GitHub] zeppelin issue #2957: [ZEPPELIN-3441] Fix license check failure in r
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2957
Will merge it
---
[GitHub] zeppelin pull request #2957: [ZEPPELIN-3441] Fix license check failure in r
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2957
---
[GitHub] zeppelin issue #2957: [ZEPPELIN-3441] Fix license check failure in r
Posted by zjffdu <gi...@git.apache.org>.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2957
LGTM
---