Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1624
Merge to master if there're no further discussions.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1624
Thanks @karup1990 for the great improvement!
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not ha
Github user karup1990 commented on the issue:
https://github.com/apache/zeppelin/pull/1624
```
Tests in error:
ParagraphActionsIT.testEditOnDoubleClick:443 » ElementNotVisible Element
is no...`
```
Test failure not related
---
If your project is set up for it, you
Github user karup1990 commented on the issue:
https://github.com/apache/zeppelin/pull/1624
@Leemoonsoo Thanks for the suggestion. Made the change.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1624
@karup1990 sorry, it was pending status. You can now see
https://github.com/apache/zeppelin/pull/1624#pullrequestreview-8931671
---
If your project is set up for it, you can reply to this emai
Github user karup1990 commented on the issue:
https://github.com/apache/zeppelin/pull/1624
@Leemoonsoo I dont see your comments.. (as mentioned in ur last comment).
Not sure If I am missing something
---
If your project is set up for it, you can reply to this email and have your
re
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1624
@karup1990 Thanks for the great contribution.
Tested and works well. I left a comment. Please take a look!
---
If your project is set up for it, you can reply to this email and have your
r
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1624
@karup1990 Let me take a look and get back to you.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user karup1990 commented on the issue:
https://github.com/apache/zeppelin/pull/1624
Any suggestions/feedbacks ? @corneadoug @Leemoonsoo
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
Github user karup1990 commented on the issue:
https://github.com/apache/zeppelin/pull/1624
Ready for review @corneadoug @Leemoonsoo
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this featu
10 matches
Mail list logo