[GitHub] zeppelin issue #1662: [ZEPPELIN-1695] Enforce pom.xml style while removing u...

2016-11-23 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1662 Thanks! Looks great to me, will merge to master as soon as CI is green, if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1662: [ZEPPELIN-1695] Enforce pom.xml style while removing u...

2016-11-23 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1662 @bzz I couldn't consider the reviewer's point of view. I will split this! Thanks again :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #1662: [ZEPPELIN-1695] Enforce pom.xml style while removing u...

2016-11-23 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1662 @1ambda thank you for great effort! It's a bit hard to review as re-formatting is mixed with pom.xml modifications. How hard would be to have 2 separate PRs - one with improvements, and