Thanks, Damien. I did not know it, But I too notice after it happen.
On Wed, Aug 12, 2015 at 12:55 AM, Corneau Damien <[email protected]> wrote: > We need to be carefull when merging PR that has zeppelin-web changes for > some time. In PR#187 we removed the force option that allow build success > when there is jshint errors. So PR based before that point needs to rebase > (otherwise CI will be green for them, but not after merge) > On Aug 11, 2015 11:19, "madhuka udantha" <[email protected]> wrote: > > Yes, above error will be fix from PR #202 > <https://github.com/apache/incubator-zeppelin/pull/202> > > On Tue, Aug 11, 2015 at 2:44 PM, IT CTO <[email protected]> wrote: > > > Thanks, I will try the patch now. > > Eran > > > > On Tue, Aug 11, 2015 at 12:06 PM Christian Tzolov <[email protected]> > > wrote: > > > > > I believe that @leemoonsee is working on resolving this: > > > https://github.com/apache/incubator-zeppelin/pull/202 > > > If you need the patch before it is merged you can merge directly it > from > > > moon's branch: > > > > > > git remote add leemoonsoo > > > https://github.com/Leemoonsoo/incubator-zeppelin.git > > > git fetch leemoonsoo > > > git merge leemoonsoo/fix_jshint > > > > > > On 11 August 2015 at 10:55, IT CTO <[email protected]> wrote: > > > > > > > Hi, > > > > Just got the latest from master and can't build zeppelin web > > > > grunt is failing > > > > > > > > > > > > > > > > *Running 'grunt --no-color' in > > > > /home/cto/Downloads/incubator-zeppelin-master/zeppelin-web* > > > > [INFO] Running "newer:jshint" (newer) task > > > > [INFO] > > > > [INFO] Running "newer:jshint:all" (newer) task > > > > [INFO] No newer files to process. > > > > [INFO] > > > > [INFO] Running "newer:jshint:test" (newer) task > > > > [INFO] > > > > [INFO] Running "jshint:test" (jshint) task > > > > [INFO] > > > > [INFO] test/spec/controllers/notebook.js > > > > [INFO] line 11 col 72 'baseUrlSrv' is defined but never used. > > > > [INFO] line 11 col 55 'websocketMsgSrv' is defined but never > used. > > > > [INFO] > > > > [INFO] ✖ 2 problems > > > > [INFO] > > > > [INFO] Warning: Task "jshint:test" failed. Use --force to continue. > > > > [INFO] > > > > [INFO] Aborted due to warnings. > > > > [INFO] > > > > [INFO] > > > > [INFO] Execution Time (2015-08-11 08:52:21 UTC) > > > > [INFO] loading tasks 24ms ▇▇▇▇▇ 11% > > > > [INFO] newer:jshint 8ms ▇▇ 4% > > > > [INFO] newer:jshint:all 31ms ▇▇▇▇▇▇▇ 15% > > > > [INFO] newer:jshint:test 5ms ▇▇ 2% > > > > [INFO] jshint:test 139ms ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 66% > > > > [INFO] Total 210ms > > > > > > > > > > > > > > > > -- > > > Christian Tzolov <http://www.linkedin.com/in/tzolov> | Solution > > Architect, > > > EMEA Practice Team | Pivotal <http://pivotal.io/> > > > [email protected]|+31610285517 > > > > > > > > > -- > Cheers, > Madhuka Udantha > http://madhukaudantha.blogspot.com > -- Cheers, Madhuka Udantha http://madhukaudantha.blogspot.com
