Github user Savalek commented on the issue:
https://github.com/apache/zeppelin/pull/3235
@felixcheung, i managed to fix the tests.
---
Github user Savalek commented on the issue:
https://github.com/apache/zeppelin/pull/3235
@felixcheung, I'm trying to fix the tests but I can't even run `npm run
test` in `zeppelin-web` because the file `webpack.config.js` made for version
`webpack 1` but the current version `4`.
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/3235
Can you please fix the tests then. We canât merge anything that will
cause all PRs to fail in Travis.
Even when they âshouldâ be
---
Github user Savalek commented on the issue:
https://github.com/apache/zeppelin/pull/3235
@felixcheung, these tests should fall, because `maven` incorrectly handled
errors when testing through `npm`. This PR is only the first part of fixing
problems. The second part is the correction
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/3235
as I said, travis is failing with your fix? please check this test failed?
https://travis-ci.org/TinkoffCreditSystems/zeppelin/jobs/458386760
---
Github user Savalek commented on the issue:
https://github.com/apache/zeppelin/pull/3235
Any progress here?
---