[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-29 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-215931366 @Leemoonsoo thanks for the explanation! I'm +1 for your version of the description. Just updated the file based on your comment. I thi

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-28 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-215632086 @Leemoonsoo I update the file with some description and links, you would please have a look at it? --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: Fix typos in docs

2016-04-21 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/852 Fix typos in docs ### What is this PR for? 1. Fix some typos in docs. 2. Remove trailing white spaces for each line. 3. Remove leading white spaces if a line contains no

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-799] Modify `index.html...

2016-04-21 Thread cyhsutw
Github user cyhsutw closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/851 --- 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 feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-799] Modify `index.html...

2016-04-21 Thread cyhsutw
GitHub user cyhsutw reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/851 [ZEPPELIN-799] Modify `index.html` and `karma.conf.js` for `mode-python` ### What is this PR for? Python edit mode was added in #825 thank to @astroshim. I think

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-799] Modify `index.html...

2016-04-21 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/851 [ZEPPELIN-799] Modify `index.html` and `karma.conf.js` for `mode-python` ### What is this PR for? Python edit mode was added in #825 thank to @astroshim. I think we&#x

[GitHub] incubator-zeppelin pull request: Remove files under /notebook/

2016-04-21 Thread cyhsutw
Github user cyhsutw closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/847 --- 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 feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: Remove files under /notebook/

2016-04-21 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/847#issuecomment-213247192 Just checked and it *does* delete the template tutorial. Sorry for the misunderstanding. I'm closing this now, but I think maybe we can creat

[GitHub] incubator-zeppelin pull request: Remove files under /notebook/

2016-04-21 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/847#issuecomment-213200879 Thanks for the information, I'll check it later today. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] incubator-zeppelin pull request: Remove files under /notebook/

2016-04-21 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/847 Remove files under /notebook/ ### What is this PR for? We ignore files under `/notebook/` but there are some existing files under that directory. I execute the

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-21 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-212942464 Hi all, I've made some changes according to @jeffsteinmetz's comment. It would be appreciate if you could review my work again. -

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-21 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-212929545 @jeffsteinmetz just added the option as you suggested. @darionyaphet I think maybe we can make it another PR after this is merged. --- If

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-15 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/839 Improve README.md ### What is this PR for? Update README.md - Unify code block styles (fenced code blocks). - Add syntax highlighting in some code blocks

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/838#issuecomment-210702820 @felixcheung Nice idea, I'll close this one and open another instead. --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread cyhsutw
Github user cyhsutw closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/838 --- 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 feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/838 Fix unwanted link in README.md ### What is this PR for? Fix an unwanted link in `README.md` by escaping the link syntax. ### What type of PR is it? Documentation