[GitHub] zeppelin issue #2804: [DISCUSS] semicolons in javascript

2018-02-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2804 should this go to `dev@`? ---

[GitHub] zeppelin issue #2804: [DISCUSS] semicolons in javascript

2018-02-15 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2804 @mebelousov I did not follow your comment. ---

[GitHub] zeppelin issue #2804: [DISCUSS] semicolons in javascript

2018-02-15 Thread mebelousov
Github user mebelousov commented on the issue: https://github.com/apache/zeppelin/pull/2804 @prabhjyotsingh it's great that one of PMC file such PR. A lot of tests show that JS works faster without syntax warnings. :+1: ---

[GitHub] zeppelin issue #2804: [DISCUSS] semicolons in javascript

2018-02-14 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2804 cc: @1ambda @AhyoungRyu @r-kamath ---