Github user bzz commented on the issue:

    https://github.com/apache/zeppelin/pull/1362
  
    @vensant thank you for contributing improvement!
    
    With GUI changes, in my experience, adding simple screenshots to the PR 
itself (drag'n'drop to description) before\after usually helps to speed up the 
review process a lot.
    
    CI failure seems not related, look lik networking issue on Travis side :\
    
    ```
    [INFO] Zeppelin: web Application .......................... FAILURE [01:20 
min]
    [INFO] grunt-ng-annotate@0.10.0 node_modules/grunt-ng-annotate
    [ERROR] npm ERR! argv 
"/home/travis/build/apache/zeppelin/zeppelin-web/node/node" 
"/home/travis/build/apache/zeppelin/zeppelin-web/node/node_modules/npm/bin/npm-cli.js"
 "install" "--color=false"
    [ERROR] npm ERR! node v0.12.13
    [ERROR] npm ERR! npm  v2.15.0
    [ERROR] 
    [ERROR] npm ERR! Callback called more than once.
    [ERROR] npm ERR! 
    [ERROR] npm ERR! If you need help, you may report this error at:
    [ERROR] npm ERR!     <https://github.com/npm/npm/issues>
    [ERROR] 
    [ERROR] npm ERR! Please include the following file with any support request:
    [ERROR] npm ERR!     
/home/travis/build/apache/zeppelin/zeppelin-web/npm-debug.log
    ```


---
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