[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 


---