[GitHub] incubator-predictionio issue #297: Update document
Github user djKooks commented on the issue: https://github.com/apache/incubator-predictionio/pull/297 @dszeto I reviewed on documents, but only the part that I fixed shows the problem, and other parts used angle bracket shows fine. You can find the problem in link below. http://predictionio.incubator.apache.org/templates/recommendation/quickstart/ I'm not sure to remove all angle bracket because most of them are working fine, and if I start changing, lots of part need to be changed and maybe some one could be confused in it. Maybe it would be to let it in current state because it is pretty small bug. I'll think about other way to show it clear without replacing it. --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-predictionio issue #297: Update document
Github user dszeto commented on the issue: https://github.com/apache/incubator-predictionio/pull/297 @djKooks No problem. Please keep updating. Thanks for your contribution! --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-predictionio issue #297: Update document
Github user djKooks commented on the issue: https://github.com/apache/incubator-predictionio/pull/297 @dszeto Okay thanks! I'll keep update it if there is no problem. --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-predictionio issue #297: Update document
Github user dszeto commented on the issue: https://github.com/apache/incubator-predictionio/pull/297 Thanks for your contribution! I don't have a problem with replacing angle brackets with square brackets. The only ask from me is to keep it consistent throughout the whole documentation. Thanks! --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-predictionio issue #297: Update document
Github user djKooks commented on the issue: https://github.com/apache/incubator-predictionio/pull/297 One more, while I am working on tutorial, there are some broken parts in document that are using angle bracket. It looks it is hard to control on markdown text. http://stackoverflow.com/questions/20610244/how-to-use-angle-brackets-in-url-on-github-flavored-markdown How about changing this to square bracket? This is a suggestion, so please let me know if this is not a good choice. --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
