Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/534#issuecomment-164755385
  
    @bzz Of course, the code review will come :)
    
    Well, JIRA issue is JIRA issue, PR Template is PR Template :)
    You will find that there is a lot of nice categories and questions that 
will help review the PR.
    
    One good example is:
    >Does the licenses files need update?
    
    Which answer in this PR's case would be: Yes
    
    But ``angular-resource`` was not added to the license file


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to